Skip to content

fix(desktop): complete request access flow#2073

Merged
wesbillman merged 3 commits into
mainfrom
pinky/request-access-join
Jul 18, 2026
Merged

fix(desktop): complete request access flow#2073
wesbillman merged 3 commits into
mainfrom
pinky/request-access-join

Conversation

@wesbillman

@wesbillman wesbillman commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a community URL field to the first-community request-access screen
  • connect through the existing community onboarding transaction after the host adds the user's npub
  • check membership before profile setup and show a retryable not-yet-added error

Testing

  • pnpm --dir desktop check
  • pnpm --dir desktop build
  • pnpm --dir desktop typecheck
  • pnpm --dir desktop test (3,156 tests)
  • full pre-push hook (desktop, Rust, Tauri, and mobile suites)

Screenshot

Request access screen with community URL input

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman
wesbillman requested a review from a team as a code owner July 18, 2026 17:01
wesbillman and others added 2 commits July 18, 2026 11:05
Align the public key card and community URL controls more closely with the onboarding mock.

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
Bring the branch onto the current CI-tested base after overlapping onboarding changes landed.

Co-authored-by: Pinky <44b8e82baa6e0e254e0208d68f335c283c94e7b78dd1fa10d5a49d3f13dd0435@sprout-oss.stage.blox.sqprod.co>
Signed-off-by: Wes <wesbillman@users.noreply.github.com>
@wesbillman
wesbillman merged commit 426497a into main Jul 18, 2026
25 checks passed
@wesbillman
wesbillman deleted the pinky/request-access-join branch July 18, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant