Skip to content

feat: add optimistic updates for deleting and changing hidden status of event types.#4721

Merged
zomars merged 7 commits intocalcom:mainfrom
G3root:event-types-optimistic-updates
Oct 18, 2022
Merged

feat: add optimistic updates for deleting and changing hidden status of event types.#4721
zomars merged 7 commits intocalcom:mainfrom
G3root:event-types-optimistic-updates

Conversation

@G3root
Copy link
Copy Markdown
Contributor

@G3root G3root commented Sep 27, 2022

What does this PR do?

https://www.loom.com/share/8315c31c1c6840338ec918929fc3194c

Environment: Staging(main branch) / Production

Type of change

  • New feature (non-breaking change which adds functionality)

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 27, 2022

@G3root is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 6, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Oct 12, 2022 at 8:41AM (UTC)

Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure we can improve the legibility a little bit by extracting the update state logic into a function. But it works for now.

Ship it

@zomars zomars merged commit 7f2560e into calcom:main Oct 18, 2022
@G3root G3root deleted the event-types-optimistic-updates branch October 19, 2022 03:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants