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.

Sign up for GitHub

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

Jump to bottom

Hide logout option from CHPs #1685

Closed
alxndrsn opened this issue Nov 25, 2015 · 5 comments
Closed

Hide logout option from CHPs #1685

alxndrsn opened this issue Nov 25, 2015 · 5 comments
Labels
Priority: 1 - High Blocks the next release. Type: Feature Add something new Won't fix: Ancient Too old to investigate
Milestone
Backlog: Depriori...

Comments

@alxndrsn
Copy link
Contributor

No description provided.

@alxndrsn alxndrsn added 1 - Scheduled Type: Feature Add something new Priority: 1 - High Blocks the next release. labels Nov 25, 2015
@garethbowen
Copy link
Member

Whether or not we hide the logout action, under the current design all CHPs will have to log in again at some point. If you don't bother training them on how to do this, or worse still, don't provide them with their username and password then they will have to call an administrator to get instructions. I strongly disagree with this change.

@garethbowen
Copy link
Member

@joshnesbit mentioned in Slack that the only issue for CHPs was accidentally hitting the logout button. Given this it seems to me that a confirm dialogue would solve this issue rather than removing the button altogether.

@alxndrsn alxndrsn self-assigned this Nov 27, 2015
alxndrsn added a commit that referenced this issue Nov 27, 2015
@alxndrsn
Hide logout option from CHPs
a3a16a0 
Issue: #1685
@alxndrsn
Copy link
Contributor Author

@garethbowen please review. Hopefully this change is not too intrusive and can easily be reversed when we find a more pleasing approach.

@alxndrsn alxndrsn assigned garethbowen and unassigned alxndrsn Nov 27, 2015
@alxndrsn
Copy link
Contributor Author

@abbyad this will require suitable extra permissions to be set up (i.e. cannot_logout)

@alxndrsn alxndrsn assigned mandric and unassigned garethbowen Dec 1, 2015
@ghost ghost modified the milestones: Mobile application overflow, Iteration 2015-45: Field Test Preparation and QA Dec 1, 2015
@mandric
Copy link
Contributor

mandric commented Dec 2, 2015

Work was continued for this elsewhere: #1730

@mandric mandric closed this as completed Dec 2, 2015
@mandric mandric removed their assignment Dec 2, 2015
@garethbowen garethbowen added Won't fix: Ancient Too old to investigate and removed 4 - Acceptance Testing labels Sep 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority: 1 - High Blocks the next release. Type: Feature Add something new Won't fix: Ancient Too old to investigate
Projects
None yet
Milestone
Backlog: Deprioritized Mobile Applica...
Development

No branches or pull requests
3 participants
@mandric @alxndrsn @garethbowen