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

Web -Chat - Extra rooms: admin and announce displayed after log in with owner account #11281

Closed
kbecciv opened this issue Sep 26, 2022 · 12 comments
Assignees

Comments

@kbecciv
Copy link

kbecciv commented Sep 26, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Issue found when execute PR #10491

Action Performed:

  1. Go to staging.new.expensify.com
  2. Create 3 accounts:
    applausetester+workspace-admin@applause.expensifail.com/Feya86Katya
    applausetester+workspace-employee@applause.expensifail.com/Feya86Katya
    applausetester+workspace-owner@applause.expensifail.com/Feya86Katya
  3. Login with in OldDot with applausetester+workspace-owner@applause.expensifail.com
    and create workspace inviting applausetester+workspace-employee@applause.expensifail.com
    as an employee and applausetester+workspace-admin@applause.expensifail.com as an admin.
  4. Log in NewDot with applausetester+workspace-owner@applause.expensifail.com
    account and check 3 policy expense chats, one for each workspace member, and two rooms: #announce and #admin

Expected Result:

3 policy expense chats, one for each workspace member, and two rooms: #announce and #admin are displayed with owner account

Actual Result:

Extra rooms: admin and announce displayed after log in with owner account

Workaround:

Uknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.2.6.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation
PR1

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

@melvin-bot
Copy link

melvin-bot bot commented Sep 26, 2022

Triggered auto assignment to @stitesExpensify (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@luacmartins
Copy link
Contributor

@aldo-expensify maybe it's related to #10491?

@roryabraham
Copy link
Contributor

I'm confused by this / not able to reproduce it:

Login with in OldDot with applausetester+workspace-owner@applause.expensifail.com
and create workspace inviting applausetester+workspace-employee@applause.expensifail.com
as an employee and applausetester+workspace-admin@applause.expensifail.com as an admin.

How is it possible to add people to a workspace from OldDot, since clicking on the workspace brings you to NewDot? From NewDot, it's not yet possible to invite people to a workspace with a specific role (admin vs employee). So I'm confused exactly what was done here to create the state we see in the screenshot above.

I did the following:

  1. Created three accounts, workspace-owner, workspace-admin, workspace-employee
  2. Signed into OldDot as workspace-owner
  3. Created a Free Plan (aka Workspace). This took me to NewDot and signed me in.
  4. Invited workspace-admin and workspace-employee to the policy from NewDot.

Almost? Everything seems correct now:

workspace-owner

image

workspace-admin

image

  • Missing #admins room because they are not an admin and cannot be made an admin.
    • Unless I'm mistaken, this is not a bug but just a missing feature
  • Missing workspace chat 🐛

workspace-employee

Same state as workspace-admin

@roryabraham
Copy link
Contributor

In any event, workspace chats are still behind a beta, so we don't need to block deploy on this

@roryabraham
Copy link
Contributor

cc @trjExpensify @JmillsExpensify Can you buddy-check my above analysis of this issue please?

@JmillsExpensify
Copy link

Agree we don't need to block a deploy.

@JmillsExpensify
Copy link

JmillsExpensify commented Sep 26, 2022

For the workspace chats, can you check whether they were created (via chat switcher), just potentially don't show proactively in the LHN?

@roryabraham
Copy link
Contributor

can you check whether they were potentially created, just potentially don't show proactively in the LHN?

Yes. I checked and they were not discoverable via search or request money pages

@roryabraham
Copy link
Contributor

However, when I sent a message in the workspace chat (as the workspace-owner), then the workspace chat showed up for employee and admin

@roryabraham
Copy link
Contributor

Some more details over here: #10491 (comment)

@aldo-expensify
Copy link
Contributor

aldo-expensify commented Sep 26, 2022

Thanks for testing @roryabraham

I understand that the workspace-employee (same as workspace-admin at the moment) should only see their policy expense chat, but I think this hides away if it is empty and archived. I'll test again putting messages on each chat/room before closing the workspace. What I think is a bug is that the #announce room should have disappeared for all non-admin/non-owner

@roryabraham
Copy link
Contributor

In any event, I think we should close this issue because I'm not able to reproduce it as written.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants