search

socketio / socket.io-admin-ui

Admin UI for Socket.IO
https://admin.socket.io
MIT License
358 stars 95 forks source link

logout feature #12

Open hasan-4 opened 3 years ago

hasan-4 commented 3 years ago

add logout feature which will clear the browser local storage \ \ \ Thank you

hasan-4 commented 3 years ago

add this code with logout button

localStorage.clear(); // it will clear the  browser local storage data only
fahadpf commented 3 years ago

I would not suggest clearing the whole localStorage. I'm using this project on a subdirectory of my domain, and I also have some very important data stored in localStorage.

It would be better if we clear only those keys which are set by Admin UI.

hasan-4 commented 3 years ago

@fahadpf i agree with you

Replay: @mdawaffe

I would not suggest clearing the whole localStorage. I'm using this project on a subdirectory of my domain, and I also have some very important data stored in localStorage.

It would be better if we clear only those keys which are set by Admin UI.

darrachequesne commented 3 years ago

That's a good idea. @imithu would you have the time to implement it?

hasan-4 commented 3 years ago

sure, i will do it in nuxt

Replay: @darrachequesne That's a good idea. @imithu would you have the time to implement it?

Thomas-1985 commented 2 years ago

It would be really great if this could be implemented

darrachequesne commented 2 years ago

@Thomas-1985 could you please explain your use case?

I'm not sure about the feature that is requested here. Should we overload the "update" button?

Thomas-1985 commented 2 years ago

Well there should be a "logout" button that logs you out of the ui. Like @fahadpf mentioned, the code should only clear the set localStorage keys and not clear it out completely (there could be other data stored).

tfrancois commented 1 year ago

This excellent piece of software is sorely lacking this feature - it would make this product almost PERFECT. Please implement.