Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: replace toaster with sonner #19342

Open
wants to merge 61 commits into
base: main
Choose a base branch
from

Conversation

nayan-bagale
Copy link
Contributor

@nayan-bagale nayan-bagale commented Feb 18, 2025

What does this PR do?

/claim #19238

Before:
https://github.com/user-attachments/assets/3d90fd1d-c88b-4321-8770-6bdd9478b348

After:
https://github.com/user-attachments/assets/75a2e300-7941-49fb-b198-cb70af668c2a

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

Removed react-hot-toast and Added sonner to

  • /apps/web/package.json
  • /packages/lib/package.json

nayan-bagale and others added 30 commits February 11, 2025 22:56
* perf: Move PlainChat up the hierarchy to remove rerenders

* rmeove yarn lock

---------

Co-authored-by: Nizzy <[email protected]>
Co-authored-by: nizzy <[email protected]>
* feat: Use new /routing URL for routing forms

* Fixes

* Fix bug

* precent crash

---------

Co-authored-by: Hariom Balhara <[email protected]>
* refactor: version old slots

* feat: init new slots endpoints

* chore: range format

* fix: duratin

* fix: duratin

* test: slot releaseAt

* refactor: reserve slot response

* refactor variable name

* docs: have new slots controller in docs

* feat: crud for slots reservations

* refactor: use exclude all for response

* docs

* chore: slots input service

* refactor mini

* refactor: remove unused imports

* docs

* handle orgSlug for dynamic events

* refactor: correct name

* docs

* add optional organizationSlug to BySlug search

* refactor: slot output format

* refactor: return seated slot info

* docs

* rename functions

* refactor: slots seated response

* fix: handle same username in org and non org

* refactor: test format

* fix: allow reservationDuration only for authed requests

* chore: check if auth user has permissions to define custom reservation duration

* refactor: split e2e test files

* refactor: invalid request exception codes

* chore: old slots controller uses all old versions
* refactor: remove depreceated fields

* chore: use v2 endpoint

* chore: update V2 Api type

* chore
…om#19286)

* remove getServerSideProps

* refactor teams route
* Throw error if no contact is passed to createEvent

* chore: update error message

* chore

---------

Co-authored-by: Udit Takkar <[email protected]>
Co-authored-by: Udit Takkar <[email protected]>
* default people filter for admin

* chore

* type check

* added test case

* made test concise to reduce flakiness

* add timeout to wait for second get response before counting

* update timeout

---------

Co-authored-by: Peer Richelsen <[email protected]>
* init

* test: v2 bookings returns rating

---------

Co-authored-by: supalarry <[email protected]>
Co-authored-by: Lauris Skraucis <[email protected]>
…vailabil… (calcom#19263)

* Reapply "fix: Multiple rr-hosts combine to create erroneous availability (calcom#18772)"

This reverts commit 5dbc6d9.

* Only thing left is a diff to show which slots are removed

* Add logging around time slot results

* Setup logging through axiom

* .... forgot

* fix: type errors

* Only log when there are differences

* Updated from old which had a failing test to new logic
* fix: flaky test

* fix another test

* update

---------

Co-authored-by: Eunjae Lee <[email protected]>
Co-authored-by: unknown <[email protected]>
* add UpgradeTipWrapper

* refactor insights/layout.tsx

---------

Co-authored-by: Anik Dhabal Babu <[email protected]>
* feat: Use Folders to organize SMS links on Dub

* Update yarn.lock

---------

Co-authored-by: sean-brydon <[email protected]>
Co-authored-by: Bailey Pumfleet <[email protected]>
* fix : removing all members toggle for fh

* ux enhancements

* adding placeholder
retrogtx and others added 3 commits February 18, 2025 22:27
* fix: EditLocationDialog footer positioning and structure

* fix: make RescheduleDialog footer layout and button structure right

* fix: input dialogue glitch now works great

* made tests happy

---------

Co-authored-by: Peer Richelsen <[email protected]>
@nayan-bagale nayan-bagale requested a review from a team as a code owner February 18, 2025 17:26
Copy link

vercel bot commented Feb 18, 2025

@nayan-bagale is attempting to deploy a commit to the cal-staging Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added ✨ feature New feature or request 💎 Bounty A bounty on Algora.io 💻 refactor 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Feb 18, 2025
@CLAassistant
Copy link

CLAassistant commented Feb 18, 2025

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
15 out of 16 committers have signed the CLA.

✅ Udit-takkar
✅ nayan-bagale
✅ supalarry
✅ joeauyeung
✅ hbjORbj
✅ eunjae-lee
✅ vijayraghav-io
✅ steven-tey
✅ retrogtx
✅ anikdhabal
✅ emrysal
✅ Amit91848
✅ Ryukemeister
✅ ThyMinimalDev
✅ sanchitttt
❌ TusharBhatt1
You have signed the CLA already but the status is still pending? Let us recheck it.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 18, 2025
@graphite-app graphite-app bot requested a review from a team February 18, 2025 17:27
@dosubot dosubot bot added the ui area: UI, frontend, button, form, input label Feb 18, 2025
@regexyl
Copy link
Contributor

regexyl commented Feb 18, 2025

@nayan-bagale (you might have tagged the wrong person, I believe you meant @retrogtx) xD

Copy link

graphite-app bot commented Feb 18, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (02/18/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add community label" took an action on this PR • (02/18/25)

1 label was added to this PR based on Keith Williams's automation.

@retrogtx
Copy link
Contributor

I thought you closed the PR because it was annoying to resolve the merge conflicts

Thank you for the PR, this works!

@PeerRich PeerRich enabled auto-merge (squash) February 19, 2025 09:02
Copy link

socket-security bot commented Feb 19, 2025

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report↗︎

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/@formkit/[email protected]

View full report↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙋 Bounty claim 💎 Bounty A bounty on Algora.io community Created by Linear-GitHub Sync ✨ feature New feature or request 🧹 Improvements Improvements to existing features. Mostly UX/UI 💻 refactor ui area: UI, frontend, button, form, input
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CAL-5152] chore: replace toaster with sonner