search

mlool / workday-calendar-extension

MIT License
58 stars 23 forks source link

Sync Worklist To Saved Schedules #130

Closed jaskirat-gill closed 3 months ago

jaskirat-gill commented 3 months ago

What Issue does this PR resolve? (Link to GitHub Issue, approved features and bugs will be given priority) #108

Please provide a video demo below, or a screenshot and description of the change.

https://github.com/mlool/workday-calendar-extension/assets/60302907/3a1372e0-6400-4dba-9eed-7fe32c7bc0d6

https://github.com/mlool/workday-calendar-extension/assets/60302907/25b4c034-86f3-4727-842f-754310276450

Tag reviewers for the PR below.

jaskirat-gill commented 3 months ago

@ThatNerdSquared thanks for the review. Implemented most of the feedback, there was actually not a lot that can be done for SyncSavedSchedules as it must be a React Component to use React hooks (useContext for the Modals) but still cleaned it up best I could