-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Chat - Hidden is not shown when starting new chat with a user without previous conversation #33901
Comments
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Triggered auto assignment to @NikkiWines ( |
Not reproduced on Production hiddenprod.mp4 |
I'm unable to reproduce this on staging v1.4.21-1 in MacOS Chrome. @kbecciv can you confirm this is reproducible on staging? Screen.Recording.2024-01-03.at.14.33.34.mov |
@NikkiWines I can reproduce on staging v1.4.21-1 Recording.5802.mp4 |
Weird, ok! Thank you for double checking! I'll continue to dig into it |
Couldn't reproduce in dev & staging either 🤔 |
Seems like this might be tied to #32613? @puneetlath do you know of anything that went to staging recently that might be causing this? |
I think you're likely able to see that user's personal details because you were the one that invited them @kbecciv. |
We did make some back-end changes, but I don't think this is a bug. |
Sounds good, removing the blocker label then. Thanks @puneetlath! |
@kbecciv was this bug a result of a regression test? I think we might need to update the test as the scenario here, the user should actually "know" the other user and thus not see "hidden". |
@puneetlath Correct, the bug a result of a regression test, please attach TR https://expensify.testrail.io/index.php?/tests/view/4151387 |
@puneetlath Eep! 4 days overdue now. Issues have feelings too... |
@lanitochka17 @kbecciv can we just remove that test? It's no longer relevant with the changes we're making. |
Bumped Maria bout removing the test here: https://expensify.slack.com/archives/C9YU7BX5M/p1707322780645939 |
Deleted the TC (https://expensify.testrail.io/index.php?/cases/view/2778068) that covered this flow. |
Thanks @isagoico! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.4.21-1
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
Hidden should be shown as the name and on LHN
Actual Result:
Hidden is not shown but email is shown
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6331563_1704308267587.nothidden.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: