-
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
[$250] Workspace - Added non-existent members in WS disappear from the list. #40301
Comments
Triggered auto assignment to @jliexpensify ( |
We think that this bug might be related to #wave-collect - Release 1 |
@jliexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
@lanitochka17 my staging version is v1.4.61-8 and I can't repro this. Going to see if C+ can. |
Job added to Upwork: https://www.upwork.com/jobs/~012892de41a5162e21 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
Hi @mananjadhav - any chance you can repro this? My staging is not 1.4.62-10, is yours? Thanks! |
We're working on "Allow arbitrary users to be added/removed to/from workspace chat via @mention or members list" and I'm facing the same issue. I believe it has to do with the server response. @jasperhuangg what do you think? |
Thanks @brunovjk - assigning @jasperhuangg for his eyes on this issue. |
Hmm yes, this does seem like a back-end issue, it appears the workspace members page skips showing users that don't have personal details set in Onyx.
I can get a PR up to fix this. |
Current assignee @mananjadhav is eligible for the Internal assigner, not assigning anyone new. |
Hmm, it appears we are sending a personal details update, but the issue is that we don't consider the users that were created as "known" to the current user. I'm unable to reproduce the issue where the users just disappear from workspace members list on dev, but I'm able to reproduce it on staging. |
Very strange, it appears when adding a single non-existing user, it works, but if you try to add multiple non-existent users, then |
@brunovjk @mananjadhav @jliexpensify There seem to be two issues at play here:
I raised a PR to fix the latter, but I wasn't able to reproduce the former in dev. Let's wait to see after my PR is deployed to staging to see if anything changes. |
This comment was marked as outdated.
This comment was marked as outdated.
@brunovjk thanks for that. You are able to reproduce because your dev environment points to our staging servers. Might be another issue that I'll look into. For now let's wait for my changes to hit staging to see if you can still reproduce. |
Nice 🙂 thank you 🚀 |
Just retested and the issue seems to have been fixed on staging when https://github.com/Expensify/Web-Expensify/pull/41771 went out |
I think we can close this out once that PR hits prod |
I can no longer reproduce. |
Thanks for checking @brunovjk - going to close this one now. Job removed from Upworks. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 1.4.62-10
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
Added non-existent members in WS should be displayed in the list of members with their login and work chats with them should be displayed by their login
Actual Result:
Added non-existent members in WS disappear from the list of members, go to "Hidden" status after update, work chats with them are shown as "Chat Report" until you update them
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6451137_1713270950299.Recording__45.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: