cal.com
cal.com copied to clipboard
calcom
feat: autosave availability after making changes
What does this PR do?
Fixes #14591
https://github.com/calcom/cal.com/assets/81948346/74cc516d-124c-4ab2-a1cd-1f3da62d95e1
@anikdhabal is attempting to deploy a commit to the cal Team on Vercel.
A member of the Team first needs to authorize it.
![vercel[bot] avatar](https://avatars.githubusercontent.com/in/8329?v=4 "Published by a pub.dev verified publisher"){kind=small}
Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
Graphite Automations
"Add community label" took an action on this PR • (05/01/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 • (05/01/24)
1 reviewer was added to this PR based on Keith Williams's automation.
![graphite-app[bot] avatar](https://avatars.githubusercontent.com/in/158384?v=4 "Published by a pub.dev verified publisher"){kind=small}
📦 Next.js Bundle Analysis for @calcom/web
This analysis was generated by the Next.js Bundle Analysis action. 🤖
This PR introduced no changes to the JavaScript bundle! 🙌
To fix
When day e.g. "Friday" is toggled 2 confirmation messages appear instead of 1. Loom: https://www.loom.com/share/627cca63e7f44adcb3e32d63eacdbb2c
## UX thoughts 1. Should we remove the "Save" button? It has no purpose if the form is autosaved. 2. I would also simplify confirmation messages when something is saved e.g. instead of changing timezone and seeing toast "✔ Working Hours schedule updated successfully" just display "✔ Saved" - the confirmation toast messages are too long IMO.
@itsberkaya @ciaranha thoughts?
Thanks for the review, I will fix that.