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

Update delete session description to be actually correct #5707

Merged
merged 2 commits into from
Jul 25, 2023

Conversation

gewenyu99
Copy link
Contributor

What does this PR do?

Related to appwrite/sdk-for-flutter#153

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

  • (Related PR or issue)

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

Copy link
Member

@lohanidamodar lohanidamodar left a comment

Choose a reason for hiding this comment

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

suggested

@@ -1 +1 @@
Use this endpoint to log out the currently logged in user from all their account sessions across all of their different devices. When using the Session ID argument, only the unique session ID provided is deleted.
Logout the current user. Use 'current' as the session ID to logout on this device, or provide a specific session ID to target a different session.
Copy link
Member

Choose a reason for hiding this comment

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

May be, logout the user. Use current as the session ID to logout current user on this device ....?

@gewenyu99 gewenyu99 requested a review from lohanidamodar June 23, 2023 14:26
@eldadfux eldadfux merged commit 887abad into master Jul 25, 2023
@abnegate abnegate deleted the docs-delete-session branch October 25, 2023 06:28
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