fix: update deprecated deleteSession usage in Next.js tutorial#2803
fix: update deprecated deleteSession usage in Next.js tutorial#2803NairAlina wants to merge 2 commits intoappwrite:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe change updates a documentation file for a Next.js tutorial step. The logout flow example is modified to invoke Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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 Tip CodeRabbit can scan for known vulnerabilities in your dependencies using OSV Scanner.OSV Scanner will automatically detect and report security vulnerabilities in your project's dependencies. No additional configuration is required. |
What does this PR do?
Updates the deprecated usage of account.deleteSession('current') in the Next.js tutorial.
The Appwrite SDK now expects an object parameter, so the code has been updated to:
account.deleteSession({ sessionId: 'current' })
This ensures the tutorial follows the current SDK syntax and prevents confusion for developers following the guide.
Test Plan
Verified the change by checking the current Appwrite SDK documentation and confirming that deleteSession expects an object parameter.
Related PRs and Issues
Closes #2489
Have you read the Contributing Guidelines on issues?
Yes, I have read the contributing guidelines.
Summary by CodeRabbit