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

[$250] Workspace - Added non-existent members in WS disappear from the list. #40301

Closed
1 of 6 tasks
lanitochka17 opened this issue Apr 16, 2024 · 21 comments
Closed
1 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review

Comments

@lanitochka17
Copy link

lanitochka17 commented Apr 16, 2024

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:

  1. Open https://staging.new.expensify.com/
  2. Log in with a new HT account
  3. Create a new WS
  4. Add 6 non-existent users to the created WS
  5. Navigate to the LHN chat menu
  6. Go back to the list of WS members

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?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6451137_1713270950299.Recording__45.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~012892de41a5162e21
  • Upwork Job ID: 1780450224453574656
  • Last Price Increase: 2024-04-17
@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Apr 16, 2024
Copy link

melvin-bot bot commented Apr 16, 2024

Triggered auto assignment to @jliexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@lanitochka17
Copy link
Author

We think that this bug might be related to #wave-collect - Release 1

@lanitochka17
Copy link
Author

@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

@jliexpensify
Copy link
Contributor

@lanitochka17 my staging version is v1.4.61-8 and I can't repro this. Going to see if C+ can.

@jliexpensify jliexpensify added the External Added to denote the issue can be worked on by a contributor label Apr 17, 2024
@melvin-bot melvin-bot bot changed the title Workspace - Added non-existent members in WS disappear from the list. [$250] Workspace - Added non-existent members in WS disappear from the list. Apr 17, 2024
Copy link

melvin-bot bot commented Apr 17, 2024

Job added to Upwork: https://www.upwork.com/jobs/~012892de41a5162e21

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Apr 17, 2024
Copy link

melvin-bot bot commented Apr 17, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav (External)

@jliexpensify
Copy link
Contributor

Hi @mananjadhav - any chance you can repro this? My staging is not 1.4.62-10, is yours? Thanks!

@brunovjk
Copy link
Contributor

brunovjk commented Apr 17, 2024

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?

@jliexpensify
Copy link
Contributor

Thanks @brunovjk - assigning @jasperhuangg for his eyes on this issue.

@jasperhuangg
Copy link
Contributor

jasperhuangg commented Apr 22, 2024

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.

 no personal details found for policy member with accountID

I can get a PR up to fix this.

@melvin-bot melvin-bot bot removed the Overdue label Apr 22, 2024
@jasperhuangg jasperhuangg added the Internal Requires API changes or must be handled by Expensify staff label Apr 22, 2024
Copy link

melvin-bot bot commented Apr 22, 2024

Current assignee @mananjadhav is eligible for the Internal assigner, not assigning anyone new.

@jasperhuangg jasperhuangg removed External Added to denote the issue can be worked on by a contributor Help Wanted Apply this label when an issue is open to proposals by contributors labels Apr 22, 2024
@jasperhuangg
Copy link
Contributor

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.

@jasperhuangg
Copy link
Contributor

Very strange, it appears when adding a single non-existing user, it works, but if you try to add multiple non-existent users, then isKnown is returned as false so we don't send their full personal details.

@jasperhuangg
Copy link
Contributor

@brunovjk @mananjadhav @jliexpensify

There seem to be two issues at play here:

  • Invited non-existent users disappear from the members list shortly after being invited
  • Invited non-existent users show as "Hidden" in the members list after being invited

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.

@jasperhuangg jasperhuangg added the Reviewing Has a PR in review label Apr 22, 2024
@brunovjk

This comment was marked as outdated.

@jasperhuangg
Copy link
Contributor

@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.

@brunovjk
Copy link
Contributor

Nice 🙂 thank you 🚀

@jasperhuangg
Copy link
Contributor

Just retested and the issue seems to have been fixed on staging when https://github.com/Expensify/Web-Expensify/pull/41771 went out

@jasperhuangg
Copy link
Contributor

I think we can close this out once that PR hits prod

@brunovjk
Copy link
Contributor

I can no longer reproduce.

@jliexpensify
Copy link
Contributor

Thanks for checking @brunovjk - going to close this one now. Job removed from Upworks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review
Projects
None yet
Development

No branches or pull requests

5 participants