Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Group chat] Add new member after typing message and pressing enter #8349

Open
noeliaSD opened this issue Nov 21, 2022 · 5 comments · May be fixed by #17014
Open

[Group chat] Add new member after typing message and pressing enter #8349

noeliaSD opened this issue Nov 21, 2022 · 5 comments · May be fixed by #17014

Comments

@noeliaSD
Copy link
Contributor

Description

Implement the new use case agreed with design team related to group chat:

  1. The group chat is setup
  2. The user adds a new member but doesn't hit the Confirm button
  3. When the user types another message and hits enter, the new member would be added and would receive the message and the To input and controls would disappear.
@noeliaSD
Copy link
Contributor Author

Here a task to cover the new use case agreed with @benjthayer. @benjthayer could you provide designs for that or add more details on it, if necessary?

Thanks

@benjthayer
Copy link

benjthayer commented Nov 21, 2022

Thanks @noeliaSD ! This flow would just replicate what we have in chat creation.

Functionality/behaviour we have recently added for chat/group chat creation:
When the user types a message, any recipients that are currently added as lozenges in the To input (prior to the user hitting the Confirm button) are added to the chat - at the same time the To input and Confirm and Cancel buttons are removed.

We need to add this same functionality for editing chat members. In the screen recording, when 'Julia' is added to the chat and a message is sent before the Confirm button is hit, Julia should be added as a member to the chat, receive the message and the To input and Confirm and Cancel buttons should be removed.

EditGroupMembers.mov

@iurimatias
Copy link
Member

moved to 2.29 due to lack of space in this milestone

@iurimatias iurimatias modified the milestones: 2.28.0 Beta, 2.29.0 Beta Feb 16, 2024
@noeliaSD noeliaSD self-assigned this Apr 16, 2024
@noeliaSD
Copy link
Contributor Author

Assigned to myself to better analyse it!

@noeliaSD noeliaSD removed their assignment May 7, 2024
@noeliaSD noeliaSD modified the milestones: 2.29.0 Beta, 2.31.0 Beta May 7, 2024
@noeliaSD
Copy link
Contributor Author

noeliaSD commented May 7, 2024

Moved to 2.31 since in 2.30 the UI-team will be focused on wallet.

@jrainville jrainville modified the milestones: 2.31.0 Beta, 2.32.0 Beta Aug 26, 2024
@jrainville jrainville modified the milestones: 2.32.0 Beta, 2.33.0 Beta Dec 4, 2024
@jrainville jrainville self-assigned this Dec 30, 2024
@jrainville jrainville moved this from Next to In Progress in Status Desktop/Mobile Board Dec 30, 2024
jrainville added a commit that referenced this issue Dec 30, 2024
Fixes #8349

When the member selector menu is open, sending a message adds/removes the members and then sends the message. Therefore, the new member will also receive the message.
@jrainville jrainville linked a pull request Dec 30, 2024 that will close this issue
3 tasks
@jrainville jrainville moved this from In Progress to Code Review in Status Desktop/Mobile Board Dec 30, 2024
jrainville added a commit that referenced this issue Dec 31, 2024
Fixes #8349

When the member selector menu is open, sending a message adds/removes the members and then sends the message. Therefore, the new member will also receive the message.
jrainville added a commit that referenced this issue Jan 3, 2025
Fixes #8349

When the member selector menu is open, sending a message adds/removes the members and then sends the message. Therefore, the new member will also receive the message.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Code Review
Development

Successfully merging a pull request may close this issue.

5 participants