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

MM-44122, MM-54124: Group Info/Mentions bottom sheet and invite group members to channel #7714

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

calebroseland
Copy link
Member

@calebroseland calebroseland commented Dec 12, 2023

Summary

  • New Group Info bottom sheet for Group Mentions
  • user_list UI changes—styling configuration and flex layout more streamlined
  • Invite group members to channel
  • Fixes bug where GM member count limit was applied to Channel -> Add Members flow
  • Fixes bug where GM member count limit could be exceeded by continuing to select profiles after limit is reached

Ticket Link

Checklist

  • [Pending] Added or updated unit tests (required for all new features)
  • Has UI changes
  • Includes text changes and localization file updates
  • Have tested against the 5 core themes to ensure consistency between them.

Device Information

This PR was tested on: Pixel 6 Pro emulator, Android SDKv33. iPhone 15 Pro iOS 17 emulator
Testing pending: Tablet

Screenshots

CleanShot.2023-12-12.at.16.50.06.mp4

User List spacing options; revamped user_item flex layout
CleanShot 2023-12-12 at 16 52 31

CleanShot 2023-12-12 at 17 01 59@2x

Release Note

Added Group Mentions bottom sheet and ability to invite group members to a channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants