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

[CLOSED] Logout button #267

Closed
oti-tech opened this issue Jun 18, 2014 · 3 comments
Closed

[CLOSED] Logout button #267

oti-tech opened this issue Jun 18, 2014 · 3 comments

Comments

@oti-tech
Copy link

Issue by glamrock
Friday Jan 24, 2014 at 23:35 GMT
Originally opened as https://github.com/opentechinstitute/luci-theme-commotion/pull/18


Added a logout button to the basic menu and made gradients compatible with older versions of Firefox, Chrome, and IE.


glamrock included the following code: https://github.com/opentechinstitute/luci-theme-commotion/pull/18/commits

@oti-tech
Copy link
Author

Comment by areynold
Wednesday Jan 29, 2014 at 19:09 GMT


@glamrock: I assume logout testing is just click the button and make sure the auth token and permissions are revoked.

What browser versions should be tested?

@oti-tech
Copy link
Author

Comment by glamrock
Wednesday Jan 29, 2014 at 19:10 GMT


@areynold Yep! If permissions aren't revoked, then there's a larger issue. Chrome >10 and FF >17 should display fine (CSS-wise).

@oti-tech
Copy link
Author

Comment by areynold
Monday Feb 03, 2014 at 17:11 GMT


Appears to work. After logout, admin password is required even when pasting the old token back into the url.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants