-
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
[$500] Profile - Time Zone is not updated on secondary device after changing on main device #18036
Comments
Triggered auto assignment to @conorpendergrast ( |
Bug0 Triage Checklist (Main S/O)
|
I don't know why we are expecting this to work in realtime, but it would require a backend change and therefore is an |
Job added to Upwork: https://www.upwork.com/jobs/~01ad9119a009eaf79a |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @0xmiroslav ( |
This works fine for me. |
@kbecciv did we try testing on some other platform combinations to see if this is unique to Android & Windows? |
@marcaaron I can request the QA team to reproduce the issue with different devices. Let me know and I will do that. |
Yes please, let's reopen this if we can reproduce it. I am guessing this was some kind of temporary disruption because I can't reliably reproduce. |
QA team is able to reproduce this issue on build v1.3.15-4 second.timezone.mp4 |
I could reliably produce this, when changing on my OSX native client (1.3.15-0) and iOS client app (1.3.15-5). To compare, I also changed pronouns between devices (always by changing the OSX app setting first). Changing pronouns: 20 seconds, precisely, three times So, yeah, reproduced! |
@conorpendergrast @marcaaron @0xmiroslav 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! |
@marcaaron This is internal, correct? |
yeah, I am going to guess that this is related to some of the issues on staging with Pusher and multiple events stuff. I don't really have a great handle on what exactly is happening there and I'm going OOO next week so I'd probably suggest that we re-add the engineering label and get someone else to help if this is urgent. FWIW I think the 3 week warning about WAQ is a bit unreasonable since it's only been "reopened" for 2 days. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @s77rt ( |
I can take this back as C+ |
📣 @0xmiroslav 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Great, thanks @0xmiroslav ! |
We are still waiting for proposals. |
@isabelastisser can you please remove |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Waiting for proposals |
@isabelastisser, @marcaaron, @0xmiroslav Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@0xmiroslav please leave updates on any daily issues you are assigned to avoid them going overdue |
Same ^ |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (First week) |
I reproduced today but not constantly reproducible Screen.Recording.2023-10-13.at.5.10.04.PM.mov |
@marcaaron It looks like the issue can no longer be reproduced, so I will close it now. Please feel free to reopen it if you disagree; thanks! |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
and login on 2 devices
Expected Result:
Time Zone should be updated
Actual Result:
Time Zone is not updated
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.6.0
Reproducible in staging?: Yes
Reproducible in production?: Yes
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
Notes/Photos/Videos: Any additional supporting documentation
Bug6033250_video_69.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: