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

Unexpected Whisper Message Displayed in NewDot #admins Room #32074

Closed
mallenexpensify opened this issue Nov 27, 2023 · 28 comments
Closed

Unexpected Whisper Message Displayed in NewDot #admins Room #32074

mallenexpensify opened this issue Nov 27, 2023 · 28 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

@mallenexpensify
Copy link
Contributor

mallenexpensify commented Nov 27, 2023

Coming from this internal issue

INTERNAL SLACK CONVERSATION FOR BACKGROUND: https://expensify.slack.com/archives/C03U7DCU4/p1700167829911559

Action Performed:

  1. Created an account with the email [email protected].
  2. Created a new workspace within the account.
  3. Accessed the 'Setup Specialist' option in the Left Hand Navigation (LHN) under Support.
  4. Entered the NewDot #admins room.

Expected Result:

No unexpected messages should appear in the #admins room, especially relating to changes not made by the user.

Actual Result:

An unexpected whisper message indicating "updated the auto-reporting frequency from 'weekly' to 'manual'" appeared in the #admins room before any interaction from the Guide, despite no changes being made to the scheduled submit settings by the user.

Workaround:

Is there a known workaround for this issue? If yes, specify if the user can still use Expensify effectively without this being fixed. Also, confirm if the user has been informed about the workaround.

Platform:

  • Web

  • iOS

  • Android

  • Desktop App

  • Mobile Web

  • Version Number: N/A

  • Logs: Not applicable.

  • Notes/Photos/Videos: None available.

  • Is this issue reproducible in staging, production, or both? Needs confirmation.

Internal only, do not post to External repos

  • Customer affected: [email protected]
  • Admin email: N/A
  • Concierge issue: N/A
  • Policy IDs: Specify if any specific policy IDs are affected.
  • Report IDs: Specify if any specific report IDs are affected.

image

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e1dc004aac63e68a
  • Upwork Job ID: 1729266543722729472
  • Last Price Increase: 2023-11-27
@mallenexpensify mallenexpensify added the Bug Something is broken. Auto assigns a BugZero manager. label Nov 27, 2023
@mallenexpensify mallenexpensify self-assigned this Nov 27, 2023
Copy link

melvin-bot bot commented Nov 27, 2023

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

@melvin-bot melvin-bot bot added the Daily KSv2 label Nov 27, 2023
Copy link

melvin-bot bot commented Nov 27, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@mallenexpensify mallenexpensify added the Internal Requires API changes or must be handled by Expensify staff label Nov 27, 2023
Copy link

melvin-bot bot commented Nov 27, 2023

Job added to Upwork: https://www.upwork.com/jobs/~01e1dc004aac63e68a

Copy link

melvin-bot bot commented Nov 27, 2023

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

@mallenexpensify
Copy link
Contributor Author

@lschurr unassigned, forgot to add myself before the Bug label.
@fedirjh do you think this can be external?

@fedirjh
Copy link
Contributor

fedirjh commented Nov 27, 2023

@mallenexpensify It seems that you shared an image from a private repository:

Screenshot 2023-11-27 at 11 39 58 PM

Accessed the 'Setup Specialist' option in the Left Hand Navigation (LHN) under Support.

@mallenexpensify How to access this option?

@mallenexpensify
Copy link
Contributor Author

oh, interesting, it hadn't crossed my mind that the image might now show (kinda cool it doesn't). Here it is
image

I'm checking on Setup Specialist in this internal Slack post, should have an update soon

@mallenexpensify
Copy link
Contributor Author

Fedi, you need to start in OldDot then get bumped over to NewDot

I am referring to the OldDot LHN, when someone clicks setup specialist in the support LHN you of olddot it should redirect them to the admins room of NewDot

@fedirjh
Copy link
Contributor

fedirjh commented Nov 28, 2023

@mallenexpensify I got it. I was able to reproduce it consistently with new accounts, I think this should be fixed on the backend. The Whisper Message is related to this option, if you update it, a new Whisper is created in newDot, My guess is that when you create a new workspace in a brand new account, this policy reporting frequency is set to weekly

Screenshot 2023-11-29 at 12 10 13 AM
CleanShot.2023-11-28.at.23.59.41.mp4

Copy link

melvin-bot bot commented Nov 29, 2023

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

@mallenexpensify
Copy link
Contributor Author

Thanks @fedirjh , adding Engineering. I'm going to leave you assigned so that, hopefully, you'll get the C+ job for reviewing the PR.

@melvin-bot melvin-bot bot added the Overdue label Dec 2, 2023
@MariaHCD
Copy link
Contributor

MariaHCD commented Dec 4, 2023

Looks like the report action is ACTION_POLICYCHANGELOG_UPDATE_AUTOREPORTING_FREQUENCY and here's the changelog that we send: https://github.com/Expensify/Web-Expensify/blob/6831794396c449298316b766345b57ec541001c8/lib/Report/Action/PolicyChangeLog/AutoReportingFrequency.php

Next step for me would be to evaluate to narrow down (in the backend) when auto reporting frequency is changed such that it's triggering the policy changelog.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Dec 4, 2023
Copy link

melvin-bot bot commented Dec 8, 2023

@MariaHCD, @mallenexpensify, @fedirjh Whoops! This issue is 2 days overdue. Let's get this updated quick!

Copy link

melvin-bot bot commented Dec 11, 2023

@MariaHCD @mallenexpensify @fedirjh this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@MariaHCD
Copy link
Contributor

MariaHCD commented Dec 11, 2023

Found the original PR that introduced the support tab in OldDot: https://github.com/Expensify/Web-Expensify/pull/37108

Looks like the auto-reporting frequency message was being sent when a user clicks on their guide and is then redirected to the #admins room. cc: @yuwenmemon @AndrewGable I'm assuming this isn't expected, any ideas on how to fix this?

@melvin-bot melvin-bot bot removed the Overdue label Dec 11, 2023
@mallenexpensify
Copy link
Contributor Author

Looks like the auto-reporting frequency message was being sent when a user clicks on their guide and is then redirected tot the #admins room. cc: @yuwenmemon @AndrewGable I'm assuming this isn't expected, any ideas on how to fix this?

@AndrewGable @yuwenmemon, 👀 plz

@melvin-bot melvin-bot bot added the Overdue label Dec 15, 2023
Copy link

melvin-bot bot commented Dec 18, 2023

@MariaHCD @mallenexpensify @fedirjh this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

Copy link

melvin-bot bot commented Dec 18, 2023

@MariaHCD, @mallenexpensify, @fedirjh Huh... This is 4 days overdue. Who can take care of this?

Copy link

melvin-bot bot commented Dec 18, 2023

@MariaHCD @mallenexpensify @fedirjh this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

Copy link

melvin-bot bot commented Dec 18, 2023

@MariaHCD, @mallenexpensify, @fedirjh Eep! 4 days overdue now. Issues have feelings too...

@mallenexpensify
Copy link
Contributor Author

No unexpected messages should appear in the #admins room, especially relating to changes not made by the user.

@marcochavezf , are you working on a PR right now about extra text showing up in chats? If so, do you think this instance will also be fixed by that PR? Also.. please provide a link if one exists.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Dec 18, 2023
Copy link

melvin-bot bot commented Dec 22, 2023

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

@mallenexpensify
Copy link
Contributor Author

@melvin-bot melvin-bot bot removed the Overdue label Dec 23, 2023
Copy link

melvin-bot bot commented Dec 25, 2023

@MariaHCD @mallenexpensify @fedirjh this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Dec 26, 2023
Copy link

melvin-bot bot commented Dec 27, 2023

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

Copy link

melvin-bot bot commented Dec 29, 2023

@MariaHCD, @mallenexpensify, @fedirjh Eep! 4 days overdue now. Issues have feelings too...

@MariaHCD
Copy link
Contributor

MariaHCD commented Jan 1, 2024

Back from OOO and resuming looking into this. @mallenexpensify I can't seem to reproduce the issue on staging. I don't get the message from the Guide at all when testing with an expensifail account setup with a workspace on the free plan:

Screen.Recording.2024-01-01.at.6.34.18.PM.mov

Are there any other prerequisites that I need in order to reproduce the issue?

@melvin-bot melvin-bot bot removed the Overdue label Jan 1, 2024
@mallenexpensify
Copy link
Contributor Author

Not able to reproduce via a gmail address either. In NewDot I see this.
image

In OldDot I don't see a setup specialist option.
image

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
None yet
Development

No branches or pull requests

4 participants