-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix report icon logic for newly created policy rooms #7252
Conversation
LGTM |
@deetergp looks like this was merged without passing tests. Please add a note explaining why this was done and remove the |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
The tests were passing. Not sure why it got flagged. Removing the |
🚀 Deployed to production by @roryabraham in version: 1.1.33-3 🚀
|
@deetergp, please review when you get the chance.
CC: @neil-marcellini for help in testing this out on iOS/android and providing screenshots!
Details
Fixes bug that causes newly created policy rooms to crash when you click on their details. Since the rooms have
Fixed Issues
$ #7182
$ #7055
$ #7262
Tests/ QA Steps
New Room
. Fill in the options in the modal and after the new room is created, verify that you see the couch icon for the room after it is created:Tested On
Screenshots
Web/ Mobile Web
iOS
Android