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

WS – Unable to invite a new user to the WS. #37970

Closed
3 of 6 tasks
izarutskaya opened this issue Mar 8, 2024 · 26 comments
Closed
3 of 6 tasks

WS – Unable to invite a new user to the WS. #37970

izarutskaya opened this issue Mar 8, 2024 · 26 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Internal Requires API changes or must be handled by Expensify staff Weekly KSv2

Comments

@izarutskaya
Copy link

izarutskaya commented Mar 8, 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: v1.4.49-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4407731
Email or phone of affected tester (no customers): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. Log in
  3. Go to settings/WS/Members
  4. Click on Invite member button and add a new expensifail accoount

Expected Result:

New user is invited

Actual Result:

Unable to invite a new user to the WS. Error appears and user disappears from the list

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

Bug6406917_1709902732401.bandicam_2024-03-08_14-35-37-301.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0161517fef7abe3bc3
  • Upwork Job ID: 1767131506125549568
  • Last Price Increase: 2024-03-11
@izarutskaya izarutskaya added DeployBlockerCash This issue or pull request should block deployment Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Mar 8, 2024
Copy link

melvin-bot bot commented Mar 8, 2024

Triggered auto assignment to @JmillsExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Mar 8, 2024
Copy link
Contributor

github-actions bot commented Mar 8, 2024

👋 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:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Mar 8, 2024

Triggered auto assignment to @srikarparsi (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@izarutskaya
Copy link
Author

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

@izarutskaya
Copy link
Author

We think that this bug might be related to #wave6-collect-submitters
CC @greg-schroeder

@srikarparsi
Copy link
Contributor

srikarparsi commented Mar 8, 2024

Hi, I'm not able to reproduce this. @JmillsExpensify do you think you can check if it works for you without an error. If so, I believe we can close this out.

Screen.Recording.2024-03-08.at.2.21.51.PM.mov

@izarutskaya
Copy link
Author

Still reproducible
But only when you add a new account, not existing.
Thank you

Recording.2241.mp4

@srikarparsi
Copy link
Contributor

Ah got it thank you, taking a look

@srikarparsi
Copy link
Contributor

srikarparsi commented Mar 9, 2024

Screen.Recording.2024-03-08.at.9.33.18.PM.mov

Hm, I'm still not able to reproduce this. I'll try doing it with a new workspace maybe.

@srikarparsi
Copy link
Contributor

Still not able to reproduce with a new workspace.

Screen.Recording.2024-03-08.at.9.38.19.PM.mov

@izarutskaya do you think you could take a screenshot of your console if you reproduce. Specifically, the AddMembersToWorkspace command in the Network tab?

@izarutskaya
Copy link
Author

@srikarparsi still reproducible for me
build v1.4.49-3
Check please my video, is it correct tab? Thanks

Screenshot 2024-03-09 211928

Recording.2243.mp4

@melvin-bot melvin-bot bot added the Overdue label Mar 11, 2024
@srikarparsi
Copy link
Contributor

srikarparsi commented Mar 11, 2024

Hm, interestingly I'm still not able to reproduce this. Thank you for the screenshot @izarutskaya. @JmillsExpensify do you think you could check if this is happening for you if you get a chance?

@melvin-bot melvin-bot bot removed the Overdue label Mar 11, 2024
@mountiny
Copy link
Contributor

Here are the logs for the QA testers.

The error is 404 Account not found which we throw here eventually.

Its not clear to me why is that for this account and I think we have to investigate it ad fix it, but its not App deploy blocker so demoting

@mountiny mountiny added Daily KSv2 Internal Requires API changes or must be handled by Expensify staff and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Mar 11, 2024
Copy link

melvin-bot bot commented Mar 11, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0161517fef7abe3bc3

Copy link

melvin-bot bot commented Mar 11, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @eh2077 (Internal)

Copy link

melvin-bot bot commented Mar 14, 2024

@JmillsExpensify, @srikarparsi, @eh2077 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Copy link

melvin-bot bot commented Mar 18, 2024

@JmillsExpensify, @srikarparsi, @eh2077 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@srikarparsi
Copy link
Contributor

Haven't gotten to this yet. I'll try to look at this on Wednesday.

@srikarparsi
Copy link
Contributor

Demoting this to a weekly, this bug seems to be account-specific since I'm not able to reproduce it but it's occurring for the QA tester.

@melvin-bot melvin-bot bot removed the Overdue label Mar 27, 2024
@srikarparsi srikarparsi added Weekly KSv2 and removed Daily KSv2 labels Mar 27, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@srikarparsi
Copy link
Contributor

No update yet.

@melvin-bot melvin-bot bot removed the Overdue label Apr 10, 2024
@melvin-bot melvin-bot bot added the Overdue label Apr 18, 2024
@JmillsExpensify
Copy link

Same as above.

@brunovjk
Copy link
Contributor

I can no longer reproduce.

"I think this should have been fixed in https://github.com/Expensify/Web-Expensify/pull/41771"

@srikarparsi
Copy link
Contributor

I'm not able to reproduce either. @izarutskaya is this reproducible for you still?

@melvin-bot melvin-bot bot added the Overdue label May 6, 2024
@srikarparsi
Copy link
Contributor

Bump @izarutskaya

@melvin-bot melvin-bot bot removed the Overdue label May 9, 2024
@srikarparsi
Copy link
Contributor

Closing this out because I am not able to reproduce. But if anyone is, please reopen and I can look at the logs of the reproducible request.

@JmillsExpensify
Copy link

Closing this out per the comment above.

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. Engineering Internal Requires API changes or must be handled by Expensify staff Weekly KSv2
Projects
No open projects
Archived in project
Development

No branches or pull requests

6 participants