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

HIGH: [Public] [$500] Members disappear after invitation in public rooms #36378

Closed
1 of 6 tasks
kbecciv opened this issue Feb 12, 2024 · 43 comments
Closed
1 of 6 tasks

HIGH: [Public] [$500] Members disappear after invitation in public rooms #36378

kbecciv opened this issue Feb 12, 2024 · 43 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Internal Requires API changes or must be handled by Expensify staff

Comments

@kbecciv
Copy link

kbecciv commented Feb 12, 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.40.0
Reproducible in staging?: y
Reproducible in production?: y
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:

  1. Create a workspace
  2. Create a public room
  3. Invite a new member (without an Expensify account)
  4. Observe the member appearing momentarily and then disappearing
  5. Refresh the page and observe the newly invited member on the member list

Expected Result:

The invited member shouldn't flash and then disappear

Actual Result:

The members disappear

Workaround:

Unknowns

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

2024-02-22_16-08-32.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~016a8e93511c34c648
  • Upwork Job ID: 1757156885762093056
  • Last Price Increase: 2024-02-26
@kbecciv kbecciv added External Added to denote the issue can be worked on by a contributor Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 12, 2024
@melvin-bot melvin-bot bot changed the title Web - Workspace - The members disappear after I invite them, no email received [$500] Web - Workspace - The members disappear after I invite them, no email received Feb 12, 2024
Copy link

melvin-bot bot commented Feb 12, 2024

Job added to Upwork: https://www.upwork.com/jobs/~016a8e93511c34c648

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

melvin-bot bot commented Feb 12, 2024

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

Copy link

melvin-bot bot commented Feb 12, 2024

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

@kbecciv
Copy link
Author

kbecciv commented Feb 12, 2024

We think that this bug might be related #wave8-collect-admins
CC @zanyrenney

@bfitzexpensify
Copy link
Contributor

I think this likely fits better with #vip-vsb. I want to test this a little more tomorrow myself to determine priority.

@melvin-bot melvin-bot bot added the Overdue label Feb 16, 2024
@bfitzexpensify
Copy link
Contributor

OK, doing a little further digging here:

  • tried this both with a new gmail account ([email protected]) and my [email protected] testing account
  • after refreshing the page, the invited members appear on the list, but no emails are received (tested with both @gmail.com and @expensifail.com accounts)
  • however, I am part of those channels. This works both for invites originating from @expensifail.com - this is me signed in to [email protected] and I'm part of the #test room that was created via [email protected]:
image

And via @gmail.com - this is me signed in to [email protected], seeing a room I was invited to from [email protected]:

image

So, it seems like the problems here aren't specific to gmail, and are the following:

  1. Invited members only appear on the members list after refreshing the page post-invite
  2. We're not sending invite emails at all, regardless of the source (private or public email domain)

@melvin-bot melvin-bot bot removed the Overdue label Feb 19, 2024
Copy link

melvin-bot bot commented Feb 19, 2024

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@bfitzexpensify
Copy link
Contributor

Raised in the #vip-vsb room here

@bfitzexpensify bfitzexpensify changed the title [$500] Web - Workspace - The members disappear after I invite them, no email received [$500] Web - Workspace - Members disappear after invitation in public rooms Feb 22, 2024
@bfitzexpensify bfitzexpensify changed the title [$500] Web - Workspace - Members disappear after invitation in public rooms [$500][HIGH] Web - Workspace - Members disappear after invitation in public rooms Feb 22, 2024
@bfitzexpensify bfitzexpensify changed the title [$500][HIGH] Web - Workspace - Members disappear after invitation in public rooms [$500] - [HIGH] Web - Workspace - Members disappear after invitation in public rooms Feb 22, 2024
@bfitzexpensify bfitzexpensify changed the title [$500] - [HIGH] Web - Workspace - Members disappear after invitation in public rooms [$500] HIGH: Web - Workspace - Members disappear after invitation in public rooms Feb 22, 2024
@bfitzexpensify
Copy link
Contributor

Added this to the #vip-vsb project, and split out the two issues - keeping the members-disappearing sub-issue here, and taking the emails not being delivered to #37067

@brunovjk
Copy link
Contributor

brunovjk commented Feb 22, 2024

I believe there is an irregularity with the api InviteToRoom response. When we invite to a non-public room, the response is complete:

'InviteToRoom' response to a non-public room

image

However, when we invite to a public room, the backend response seems incomplete to me:

'InviteToRoom' response to a public room

image

Is this expected? Offline works normally.

@bfitzexpensify
Copy link
Contributor

@marcochavezf do you have some context on this? Looks like you were working on some InviteToRoom items in #30919

@wildan-m
Copy link
Contributor

Hi, I have a proposal posted here. It's about personalDetails filtering. Currently, we do not display members with no personal details. If we want to show them, we should consider my alternative proposal to remove the personalDetails null check in RoomMembersPage.

@melvin-bot melvin-bot bot added the Overdue label Feb 26, 2024
Copy link

melvin-bot bot commented Feb 26, 2024

@fedirjh @bfitzexpensify this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Mar 25, 2024
@quinthar quinthar changed the title [$500] HIGH: [Reliability] Members disappear after invitation in public rooms HIGH: [$500] [Reliability] Members disappear after invitation in public rooms Mar 25, 2024
@quinthar quinthar changed the title HIGH: [$500] [Reliability] Members disappear after invitation in public rooms HIGH: [Reliability] [$500] Members disappear after invitation in public rooms Mar 25, 2024
@quinthar quinthar changed the title HIGH: [Reliability] [$500] Members disappear after invitation in public rooms HIGH: [Public] [$500] Members disappear after invitation in public rooms Mar 26, 2024
@bfitzexpensify
Copy link
Contributor

Still awaiting an internal volunteer. Shared in the #vip-vsb update so it's visible for someone looking for something.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Mar 26, 2024
@bfitzexpensify
Copy link
Contributor

Same update, still waiting on a volunteer.

@melvin-bot melvin-bot bot removed the Overdue label Apr 1, 2024
Copy link

melvin-bot bot commented Apr 5, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Apr 5, 2024
@bfitzexpensify
Copy link
Contributor

Same update

@melvin-bot melvin-bot bot removed the Overdue label Apr 7, 2024
@bfitzexpensify
Copy link
Contributor

Still awaiting an internal volunteer

@melvin-bot melvin-bot bot added the Overdue label Apr 11, 2024
Copy link

melvin-bot bot commented Apr 12, 2024

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

@bfitzexpensify
Copy link
Contributor

Same status - awaiting internal engineering. This issue is still being highlighted in the #vip-vsb Spring 24 items that need to be completed, so it's not going unnoticed.

Copy link

melvin-bot bot commented Apr 15, 2024

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

1 similar comment
Copy link

melvin-bot bot commented Apr 16, 2024

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

@bfitzexpensify
Copy link
Contributor

Same status - awaiting internal engineering.

@melvin-bot melvin-bot bot removed the Overdue label Apr 16, 2024
@bfitzexpensify
Copy link
Contributor

No change here.

@bfitzexpensify
Copy link
Contributor

Closing in favour of #40459, which Youssef is assigned to. I added the details of reproduction and the likely cause from this issue into that issue.

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

No branches or pull requests

7 participants