feat: "Add an Override" calendar starts with the day specified in general settings #14663
Conversation
|
@shaik-zeeshan is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
|
|
github-actions
bot
added
booking-page
dosubot
bot
added
the
bookings
github-actions
bot
added
the
🧹 Improvements
|
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link. |
Graphite Automations"Add community label" took an action on this PR • (04/19/24)1 label was added to this PR based on Keith Williams's automation. "Add consumer team as reviewer" took an action on this PR • (04/19/24)1 reviewer was added to this PR based on Keith Williams's automation. |
📦 Next.js Bundle Analysis for @calcom/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Great work so far @shaik-zeeshan!
One thing I would like to see before merging is when editing an override, we should also use the user's weekStart
|
i have update the edit calendar to use user's weekday @joeauyeung |
There was a problem hiding this comment.
Thanks for addressing the feedback and your contribution @shaik-zeeshan
What does this PR do?
Fixes #14334
Before, Add an Override calendar in Availability page starts with Sunday
After, calendar starts with day user has specified in general settings
Mandatory Tasks