search

Spectrum-CETB / LesKollab

This is a project sharing and collaborations application
MIT License
28 stars 57 forks source link

Confirmation alert Before Logout #106

Closed arpit456jain closed 3 years ago

arpit456jain commented 3 years ago

INCLUDE ALL NECESSARY FIELDS AND DELETE FIELDS THAT ARE NOT NECESSARY.

Description

I have added a confirmation alert for logout , like when user click on logout a alert will pop up and it will ask Are You sure You want to logout there will be two options cancel or Yes , Logout

Fixes #104

Type of Change:

Delete irrelevant options.

Code/Quality Assurance Only

How Has This Been Tested?

https://user-images.githubusercontent.com/55352601/115506714-d0fe7d80-a298-11eb-869a-9288b6ca2aa3.mp4

arpit456jain commented 3 years ago

Hi @arpit456jain could you please confirm that this change wouldn't create a conflict with the PR#102? I'm asking you to do so since both the PRs have worked upon logout functionality.

Hey @ishubham21 May be it will cause merge conflicts but no problem will pull the latest code and will merge in this branch and will also upload a video for confirmation 👍

arpit456jain commented 3 years ago

Hey @ishubham21 i just cross check again in pr #102 he edit the files related to admin and i had put this feature for normal user in Explore/index.php So i don't think so there will be any merge conflicts

ishubham21 commented 3 years ago

Hey @ishubham21 i just cross check again in pr #102 he edit the files related to admin and i had put this feature for normal user in Explore/index.php So i don't think so there will be any merge conflicts

Okay! Thanks for the confirmation.