Skip to content

Afr 169 allow creating group chat - #65

Open
AlissaPepino wants to merge 10 commits into
mainfrom
AFR-169-allow-creating-group-chat
Open

AlissaPepino wants to merge 10 commits into
mainfrom
AFR-169-allow-creating-group-chat

Conversation

@AlissaPepino

@AlissaPepino AlissaPepino commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Set up the UI for creating a group chat with an additional button on the existing JoinGroupModal and new modal for creating the group. Does not yet connect to backend.

JoinGroupModal:
Screenshot 2026-04-29 at 10 06 00 PM

Create Group Modal:

  • can multiselect users (shows selected with light gray background)
  • shows count of selected members
  • submit button is disabled until a group name is entered and atleast 1 member is selected
Screenshot 2026-04-29 at 10 09 38 PM

@netlify

netlify Bot commented Apr 28, 2026

Copy link
Copy Markdown

Deploy Preview for atriacoop ready!

Name Link
🔨 Latest commit b34da59
🔍 Latest deploy log https://app.netlify.com/projects/atriacoop/deploys/69f04f8a7ce04b00085eef23
😎 Deploy Preview https://deploy-preview-65--atriacoop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ansel-rangers11 ansel-rangers11 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Summer-Children

This comment was marked as outdated.

@Summer-Children Summer-Children left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR description says this frontend hasn't been connected to the backend yet. I believe the backend should be merged first, and then the frontend which is connected to the backend should be merged. If the frontend is merged first, users will see a “Create Group” button that looks functional, but clicking it won’t do anything, which results in a confusing and broken user experience

@ansel-rangers11 This is the PR that I discussed with you.

@Summer-Children
Summer-Children dismissed their stale review May 18, 2026 07:52

Accidentally approved

@AlissaPepino

Copy link
Copy Markdown
Contributor Author

@Summer-Children @ansel-rangers11 I opened up the PR in backend as well as #72 to merge into this one. Both implement the functionality. Sorry, I should've clarified this PR was just to add the UI

…chats

Afr 169 create and fetch group chats
@netlify

netlify Bot commented Jun 27, 2026

Copy link
Copy Markdown

Deploy Preview for atriacoop ready!

Name Link
🔨 Latest commit 994b944
🔍 Latest deploy log https://app.netlify.com/projects/atriacoop/deploys/6a3f23da6675e800089c1276
😎 Deploy Preview https://deploy-preview-65--atriacoop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ansel-rangers11
ansel-rangers11 requested review from Jsofeng and removed request for JeremyKalhok, Johann235 and Khoi-Duong3 August 5, 2026 04:42
@netlify

netlify Bot commented Sep 7, 2026

Copy link
Copy Markdown

Deploy Preview for atriacoop ready!

Name Link
🔨 Latest commit 77524d2
🔍 Latest deploy log https://app.netlify.com/projects/atriacoop/deploys/6a9e183c526f09000864e561
😎 Deploy Preview https://deploy-preview-65--atriacoop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

3 participants