User tests: Successful: Unsuccessful:
Labels |
Added:
?
|
Thanks @Kubik-Rubik and @sovainfo for the feedback.
I'll look to add the logout to the list of allowed tasks.
Added the logout to the list of allowed tasks.
Thank you, now it works as expected. Thank you for providing this solution!
One more tester please!
Category | ⇒ | Libraries |
It allows you to cancel but tells you need to change password when you do, may that button should be a logout button instead.
Or remove them?
The cancel button in the site form and in the toolbar on administrator form are useless if you MUST change the password.
I think it is acceptable that the cancel button brings the user back to the page with a warning that he MUST change password.
(this is how it has worked so far)
Or make it work as a logout. Current behavior is rediculous. Don't provide buttons that don't get you anywhere.
This button has nothing to do with this PR and should be addressed in another PR. Please do not mess this PR up with changes not directly related to each other (here we have a change in the library, the button would be a change in the users component itself).
We need a second successful test to get this to RTC. Thank you!
As mentioned I am not the one that added the requirement of allowing logout, Already considered that out of scope!
Already provided the second ok for test!
Status | Pending | ⇒ | Ready to Commit |
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-08 06:46:27 |
Labels |
Removed:
?
|
Thank you for your fast reaction.
It is still not possible to logout on the profile edit page through the login module with this PR. You should add this task to the list of allowed tasks.