-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Comments
Triggered auto assignment to @lschurr ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~01e1dc004aac63e68a |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @fedirjh ( |
@mallenexpensify It seems that you shared an image from a private repository:
@mallenexpensify How to access this option? |
oh, interesting, it hadn't crossed my mind that the image might now show (kinda cool it doesn't). Here it is I'm checking on |
Fedi, you need to start in OldDot then get bumped over to NewDot
|
@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 CleanShot.2023-11-28.at.23.59.41.mp4 |
Triggered auto assignment to @MariaHCD ( |
Thanks @fedirjh , adding |
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. |
@MariaHCD, @mallenexpensify, @fedirjh Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@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! |
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? |
@AndrewGable @yuwenmemon, 👀 plz |
@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! |
@MariaHCD, @mallenexpensify, @fedirjh Huh... This is 4 days overdue. Who can take care of this? |
@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! |
@MariaHCD, @mallenexpensify, @fedirjh Eep! 4 days overdue now. Issues have feelings too... |
@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. |
@MariaHCD, @mallenexpensify, @fedirjh Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Posted in #vip-vsb |
@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! |
@MariaHCD, @mallenexpensify, @fedirjh Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@MariaHCD, @mallenexpensify, @fedirjh Eep! 4 days overdue now. Issues have feelings too... |
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.movAre there any other prerequisites that I need in order to reproduce the issue? |
Coming from this internal issue
INTERNAL SLACK CONVERSATION FOR BACKGROUND: https://expensify.slack.com/archives/C03U7DCU4/p1700167829911559
Action Performed:
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
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: