Times should be updated with local times#4268
Conversation
staysgt
left a comment
staysgt
left a comment
There was a problem hiding this comment.
looks good so far! left a few comments
| const AMorPM = (time: string, offset: number) => { | ||
| const startTime = Number(time.charAt(0) + (isTwoDigitHour(time) ? time.charAt(1) : '')); | ||
| let AMorPM = time.charAt(time.length - 2) + time.charAt(time.length - 1); | ||
|
|
||
| const flipAMPM = () => { | ||
| if (AMorPM === 'AM') return 'PM'; | ||
| return 'AM'; | ||
| }; | ||
|
|
||
| // if odd, the AM / PM should be flipped | ||
| const doFlip = Math.floor((startTime + offset - 1) / 12) % 2 === 1; | ||
| if (doFlip) AMorPM = flipAMPM(); | ||
|
|
||
| return AMorPM; | ||
| }; |
There was a problem hiding this comment.
A bunch of the am/pm's are messed up on the calendar between time zones, I think since the offset could make the number negative, that might be causing a problem
London:
SF:
wavehassman
left a comment
wavehassman
left a comment
There was a problem hiding this comment.
The bug here is just changing the labels on the availability selector. I'm pretty sure the only changes necessary are reviewTimesInCurrentTimeZone and its three call sites in EditAvailability, SingleAvailabilityView, and AvailabilityScheduleView
|
The reason for the extra changes is because it is weird to have inconsistent times. If you are scheduling an event, it should show the same as on the availability, so it makes sense to change those views/modals accordingly. |
3 participants
Changes
Hours shown on the calendar now will be updated depending on the user's time zone.
If the hours roll over to a new day, the day is also updated accordingly.
Daylight savings is also factored in too.
Test Cases
To Do
Any remaining things that need to get done
Checklist
It can be helpful to check the
ChecksandFiles changedtabs.Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.
yarn.lockchanges (unless dependencies have changed)Closes #4259