Skip to content

Add logout confirmation dialog to prevent accidental logout#1376

Merged
2 commits merged intoCode-A2Z:mainfrom
Aarya-Chaudhari:logout
Feb 1, 2026
Merged

Add logout confirmation dialog to prevent accidental logout#1376
2 commits merged intoCode-A2Z:mainfrom
Aarya-Chaudhari:logout

Conversation

@Aarya-Chaudhari
Copy link
Contributor

@Aarya-Chaudhari Aarya-Chaudhari commented Jan 31, 2026

Uploading Screenshot 2026-01-31 175822.png…

Description

This PR adds a confirmation dialog before logging out to prevent accidental session termination.

Changes Made

  • Wrapped logout action with a confirmation prompt
  • Logout now proceeds only after explicit user confirmation
  • No changes to authentication logic or routing

Expected Behavior

  • Clicking Logout shows a confirmation dialog
  • Cancel → user stays logged in
  • Confirm → logout proceeds

Scope

  • Frontend-only change
  • Minimal and isolated to sidebar logout action

Fixes #1374

@vercel
Copy link

vercel bot commented Jan 31, 2026

@Aarya-Chaudhari is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@Aarya-Chaudhari
Copy link
Contributor Author

Hello @Avdhesh-Varshney
I have raised a PR.
Please review it and let me know if any changes required.

@vercel
Copy link

vercel bot commented Feb 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
code-a2z Ready Ready Preview, Comment Feb 1, 2026 7:24am

@Avdhesh-Varshney
Copy link
Collaborator

  • Footer is not required in website after logged in.
  • For logout, implement a modal not the confirmation dialog.

@Avdhesh-Varshney
Copy link
Collaborator

  • Remove everything related to footer.
  • Update the logout by adding a modal with yes or no

@Avdhesh-Varshney Avdhesh-Varshney closed this pull request by merging all changes into Code-A2Z:main in 3bbb78b Feb 1, 2026
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.

Add logout confirmation dialog to prevent accidental logout

2 participants