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

Fix particpant related and searching behavior for default rooms. #3847

Merged
merged 7 commits into from
Jul 5, 2021

Conversation

TomatoToaster
Copy link
Contributor

@TomatoToaster TomatoToaster commented Jul 1, 2021

@HorusGoul, please review when you get the chance
CC: @yuwenmemon

Details

Stop showing participants of a defaultRoom when trying to create a new Chat, Group, Payment Request, or Bill Split.
Also make it so that when we are in Search, using the email/name of a user doesn't bring up all the defaultRooms that user is a participant in.

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/169139

Tests

Tested the QA Steps locally

QA Steps

Since this is defaultRooms related, this needs to be QA'ed internally (I think?). I'll test this when this hits staging. Just ping @TomatoToaster, if this hasn't been checked off in QA checklist.

  1. Click + and create a new chat, a new group, a new money request, and a new split bill and verify when the particpant selecting screen shows up, you do not see any defaultRooms in there.
  2. Open the search and type in policy names, domain names, and room names. Verify that they show the default rooms and that those options always come first in the search once you type something in. It should still show the recents until you type something.
  3. Verify that searching for a participant's name doesn't bring up the default rooms they are in.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@TomatoToaster TomatoToaster self-assigned this Jul 1, 2021
@TomatoToaster TomatoToaster changed the title Fix particpant related behavior for default rooms. Fix particpant related and searching behavior for default rooms. Jul 2, 2021
@TomatoToaster TomatoToaster marked this pull request as ready for review July 2, 2021 20:52
@TomatoToaster TomatoToaster requested a review from a team as a code owner July 2, 2021 20:52
@MelvinBot MelvinBot requested review from HorusGoul and removed request for a team July 2, 2021 20:52
Copy link
Contributor

@HorusGoul HorusGoul left a comment

Choose a reason for hiding this comment

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

LGTM!

@HorusGoul HorusGoul merged commit 045b394 into main Jul 5, 2021
@HorusGoul HorusGoul deleted the amal-default-rooms-no-participants branch July 5, 2021 10:56
@OSBotify
Copy link
Contributor

OSBotify commented Jul 5, 2021

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@isagoico
Copy link

isagoico commented Jul 8, 2021

@TomatoToaster Another bump to test this PR 😬

@TomatoToaster
Copy link
Contributor Author

TomatoToaster commented Jul 8, 2021

Oh sorry, I was sure I marked this one. It is working all goood.

@trjExpensify
Copy link
Contributor

Is this supposedly on staging @TomatoToaster? 🤔 (v1.0.79-1)

image

@TomatoToaster
Copy link
Contributor Author

What the. You're totally right for some reason, this logic has completely reverted. I'll look into it.

@trjExpensify
Copy link
Contributor

Thanks! Sorry to be the bringer of bad news! 😅

@TomatoToaster
Copy link
Contributor Author

Should be fixed in the PR linked above (relinked here: #4141)

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.

5 participants