khannakshat7 / Elektra

⚡ML powered Electricity Outage prediction⚡
https://elecktra.uc.r.appspot.com
MIT License
41 stars 84 forks source link

Added a confirmation alert before logout #180

Closed arpit456jain closed 3 years ago

arpit456jain commented 3 years ago

Related Issuse

Added a confirmation alert before logout

Closes: #165

Describe the changes you've made

https://user-images.githubusercontent.com/55352601/116548422-d0f22380-a911-11eb-8888-8f2b9b16ece1.mp4

Checklist:

khannakshat7 commented 3 years ago

@arpit456jain Great Contribution dude !!

khannakshat7 commented 3 years ago

@arpit456jain It is not working on Dashboard webpage as it is not inherited from base.html. Add the same code to dashboard.html

arpit456jain commented 3 years ago

@arpit456jain It is not working on Dashboard webpage as it is not inherited from base.html. Add the same code to dashboard.html

Ok 👍

arpit456jain commented 3 years ago

@khannakshat7 kindly review my PR for feedback its pending from 10 days 😅

khannakshat7 commented 3 years ago

ok I'll do that

On Fri, 30 Apr, 2021, 12:22 pm 111arpit1, @.***> wrote:

@khannakshat7 https://github.com/khannakshat7 kindly review my PR for feedback its pending from 10 days 😅

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/khannakshat7/Elektra/pull/180#issuecomment-829883060, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH3XXPCKDJF6J3QBPPQRBJLTLJHTHANCNFSM43ZYSGCQ .

arpit456jain commented 3 years ago

@arpit456jain It is not working on Dashboard webpage as it is not inherited from base.html. Add the same code to dashboard.html

its done 👍