Skip to content

feat(groups): lead New Chat with groups, push the new chat, slim the Invite People sheet - #1239

Merged
bmc08gt merged 3 commits into
mainfrom
feat/group-create-invite-sheet
Nov 9, 2026
Merged

bmc08gt merged 3 commits into
mainfrom
feat/group-create-invite-sheet

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Implements the iOS side of the spec in code-payments/flipcash-client-orchestrator#124 (private-groups.md surfaces 1-2, group-invites.md surface 1).

  • New Chat: "Create a New Group" is the first row, above Find Someone by Username. Community Chat stays last.
  • After Create: the chat is pushed over the form as a normal forward push, then about 600 ms later (a delay sized to the push, since NavigationStack has no push-finished callback) the form and New Chat are removed from beneath it without animation (AppRouter.pushForCollapse(_:) / collapseTopmostPath(toTop:)). Back from the chat goes to the Chats list. Private and community creation share the call.
  • Invite People sheet, group subjects only (plain and invite link): no Recent Chats or send path; the line "Send this link to the people you want to join your group" under the pill replaces the joined/expires line and the footnote; tiles are Copy, Share, QR Code; the group presenters moved from fullScreenCover to .sheet so it fits its content, and goes to large while the QR card shows. Profile, message and currency shares are unchanged. Analytics events are unchanged.
  • Tests: group sheet layout and order, other subjects keep Recent Chats, pushForCollapse/collapseTopmostPath. Dropped the sheetMeta test with the helper.

…Invite People sheet

New Chat lists Create a New Group first. After Create, the chat replaces the form's path in one assignment so it slides in as a push. The group Invite People sheet drops Recent Chats and its send path, shows one line under the link pill, orders its tiles Copy, Share, QR Code, and wraps its height.
@bmc08gt bmc08gt self-assigned this Nov 9, 2026
@bmc08gt
bmc08gt merged commit f7afe31 into main Nov 9, 2026
4 checks passed
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