feat(invite): show toast on collection invitation accept#895
feat(invite): show toast on collection invitation accept#895B-a-l-aj-i wants to merge 2 commits intodevfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThe changes implement an invite acceptance confirmation feature. API redirect handlers now append an Changes
Sequence DiagramsequenceDiagram
participant User
participant API as API Handler
participant Router as Next.js Router
participant Dashboard as Dashboard Component
participant Toast as Toast Service
User->>API: Accept invitation (click link)
API->>API: Validate & update collaborator
API->>Router: Redirect to /?invite=accepted
Router->>Dashboard: Render with query param
Dashboard->>Dashboard: useEffect detects invite=accepted
Dashboard->>Toast: Call successToast()
Toast->>User: Display "You've joined the shared collection"
Dashboard->>Router: router.replace() to remove param
Router->>Dashboard: Rerender (param removed, toast won't retrigger)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
show-toast-on-collection-invite-accept-1774874043028.mp4 |
Summary by CodeRabbit