-
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
Chat - Report comment date & time always shows Today for public and phone accounts #10046
Comments
Triggered auto assignment to @aldo-expensify ( |
Failed to reproduce, tested with a Tested in production and staging. |
@aldo-expensify, It got fixed by #10043. |
@thesahindia , great! thanks for letting me know :), you saved me a bunch of time . |
@aldo-expensify Still able to repro with these accounts :[email protected]/Feya87Katya Recording.825.mp4 |
@kavimuru thanks for providing those cases, I'll test again! |
yep, confirmed that it still happening with that account. I looked at the messages coming in the request from the server, and the |
There is something weird in the personal details stored for [email protected] The timezone key is referencing the personal details itself |
It seems like DB needs scrutiny. |
Yep, it is stored in the DB like that |
Reproduced locally and confirmed that the bug that caused this was fixed by the PR: #10043 We just have to take care of the bad timeZones that got saved into the database (will do that next) |
Ok, we corrected the bad |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Issue found when executing CP #10043
Action Performed:
Expected Result:
Date & Time next to usernames should be accurate
Actual Result:
Report comment date & time always shows Today for public and phone accounts
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.85.6
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): +15082211196/Feya86Katya, [email protected]/w323052760We
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Bug5657116_Recording__886.mp4
Screen_Recording_20220721-113422_Chrome.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: