-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 - The timezone does not exist after toggle off Automatically determine your location
#28329
Comments
Automatically determine your location
Automatically determine your location
Job added to Upwork: https://www.upwork.com/jobs/~015ad1c1482cffdc01 |
Triggered auto assignment to @joekaufmanexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Profile - The timezone does not exist after toggle off Automatically determine your location What is the root cause of that problem?We are missing some timezones in
What changes do you think we should make in order to solve the problem?We should add those timezones that are missed in What alternative solutions did you explore? (Optional)I think to BE should return the correct name of the timezone, example |
@izarutskaya You have missed my proposal ^^ |
@suneox's ProposalPlease re-state the problem that we are trying to solve in this issue.The timezone does not exist after toggle off What is the root cause of that problem?The list of timezone is missing from Line 1 in bb32cc5
What changes do you think we should make in order to solve the problem?We should get data from a timezone database and I have parse List of tz database time zones upload file bellow we can replace for TIMEZONES.js What alternative solutions did you explore? (Optional)Create an automation script to update timezone list from Tz_database fetch(“https://en.wikipedia.org/wiki/List_of_tz_database_time_zones”).then(r => r.text()).then(html => {
const $=cheerio.load(html);
const timezones = $(“.wikitable.sortable:first tbody tr td:nth-child(2)“).map((k,v) => $(v).text()).toArray();
// save to TIMEZONES.js
}) |
The proposal from my comment in slack is missed attachment file so I would like to correct and update my alternative proposal bellow ProposalPlease re-state the problem that we are trying to solve in this issue.The timezone does not exist after toggle off What is the root cause of that problem?The list of timezone is missing from Line 1 in bb32cc5
What changes do you think we should make in order to solve the problem?We should get data from a timezone database and I have parse List of tz database time zones upload file bellow we can replace for TIMEZONES.js What alternative solutions did you explore? (Optional)The timezone will call API App/src/libs/actions/PersonalDetails.js Line 276 in 13b2291
We can create a mapping config for timezone before submit to BE follow the tz_database backward link const timezoneBackwardMap = {
'Africa/Accra': 'Africa/Abidjan',
'Africa/Addis_Ababa': 'Africa/Nairobi',
.....
}; and update App/src/libs/actions/PersonalDetails.js Line 280 in 13b2291
to {
timezone: JSON.stringify({automatic: timezone.automatic, selected: timezoneBackwardMap[timezone.selected] || timezone.selected}),
} |
I recommended using the primary solution from my proposal we only updated the missing timezone and BE still store a user tz correctly |
I don't understand what the bug is here. This is working for me, and my sense is it is in the video as well. Discussing here. |
@joekaufmanexpensify I have explained in this comment, and you have to change your device time to another country I provide in precondition step |
Closing this based on this discussion. I can't reproduce the bug, and we don't want to proceed if it is not consistently reproducible. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Precondition:
Expected Result:
The timezone will show and auto select from "Automatically determine your location"
Actual Result:
No timezone selection
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: v 1.3.74-0
Reproducible in staging?: Y
Reproducible in production?: Y
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
002.mp4
RPReplay_Final1695832227.MP4
RPReplay_Final1695690200.MP4
RPReplay_Final1695747320.MP4
Expensify/Expensify Issue URL:
Issue reported by: @suneox
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695577882053919
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: