feat: Availability slider for orgs#10740
Conversation
Co-authored-by: Omar López <zomars@me.com>
… from our default label "Your Name" (#10618) Co-authored-by: Carina Wollendorfer <30310907+CarinaWolli@users.noreply.github.com>
…s/form/step (#10442) Co-authored-by: gitstart-calcom <gitstart@users.noreply.github.com> Co-authored-by: Keith Williams <keithwillcode@gmail.com>
Co-authored-by: Hariom Balhara <hariombalhara@gmail.com>
Co-authored-by: CarinaWolli <wollencarina@gmail.com>
* feat: mailhog fixture * fix: nodemailer to dispatch emails with e2e env * fix: remove space from email subject * feat: fixture getFirstEventAsOwner * feat: assert email subjects
* fix and enable dynamic booking test * remove page pause --------- Co-authored-by: Alex van Andel <me@alexvanandel.com>
* fix: Broken team events if a user with the same name exists * Fix tests + fix usernameList optionality
Co-authored-by: Omar López <zomars@me.com>
Co-authored-by: sean-brydon <55134778+sean-brydon@users.noreply.github.com> Co-authored-by: Omar López <zomars@me.com>
Co-authored-by: Peer Richelsen <peeroke@gmail.com>
| dateRanges: DateRange[]; | ||
| } | ||
|
|
||
| export function AvailabilitySliderTable() { |
There was a problem hiding this comment.
I had my availability set up between 9 - 4:30PM but i see 9 - 5PM
There was a problem hiding this comment.
Wait this is right isnt it? Yes youre only available for 1/2 of 4:30 but your still available in the 4 slot? I wonder if we should display an amber cell here let me try
There was a problem hiding this comment.
Screenshot above appears to be right for 9-4:30?
Granted 4 shell should probably be amber. Will play around in figma to see if we can get something to show this.
Will tackle in a follow up if you believe this isnt a blocker @Udit-takkar
There was a problem hiding this comment.
@sean-brydon this is a bit confusing because 9 - 5 also shows this same UI. the screenshot shows slots 9 - 10, 10- 11, 11-12, 12 - 13, 13- 14, 14-15, 15- 16, 16-17. the last slots should be 16:00 - 16:30. maybe because there isn't any clear boundary that help differentiate where does 4pm starts and ends. I expected 16:30 to be where "16" is displayed in the screenshot because 9:00 AM in the middle of 8 and 9 so middle of 16 and 17 would be 17:00 and not 16:30
There was a problem hiding this comment.
I understand - we need to make some compromises here. As we can't seperate into 30 min cells as we'd just get the same issue with 45 min offset timezones. @Jaibles has cleared this up in the designs for the selection hover
Udit-takkar
left a comment
There was a problem hiding this comment.
@sean-brydon Great PR 🚀
… display background color
| minuteOffsetApplied = true; | ||
| minuteOffsetStyles.marginLeft = `${DAY_CELL_WIDTH * (offset % 1)}px`; |
There was a problem hiding this comment.
yes - that's intentional it should only shift for decimal places like 3.5 and 4.75
Implements a WTB type of display for users availability and timezones for use in ORGS only
How to test:
Create a ORG
Invite a few users
go through onboarding with them and change the timezone of each user.
Visit /availability you should see a tab to switch to your team or visit /availability?type=team directly.
Compare generated hours with the worldtimebuddy equivalent.
Ensure availability matches up with the users default schedule
https://www.loom.com/share/e0508a3b048e4f4fb4a67682f4373ec1
