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

UI: UX for logging in as administrator #177

Closed
jhmarina opened this issue Oct 25, 2022 · 2 comments · Fixed by s3gw-tech/s3gw-ui#96
Closed

UI: UX for logging in as administrator #177

jhmarina opened this issue Oct 25, 2022 · 2 comments · Fixed by s3gw-tech/s3gw-ui#96
Assignees
@votdev
Labels
area/ui User Interface kind/design Design work
Milestone
v0.9.0

Comments

@jhmarina
Copy link
Contributor

jhmarina commented Oct 25, 2022 •
edited

What needs to be done

Mock up a UX that handles both the login and regular user logins.

Why it needs to be done

Currently, the UX displays a checkbox to log in as an admin. This can lead to confusion since it feels like anyone can log in as an admin.

There are different possible approaches to fix this.

Best approach from a UX perspective

Testing whether the credentials are an admin or not and then logging in is the ideal option since it allows for removing the checkbox. We may not need a mockup.

Acceptance Criteria

  • @jhmarina mocks up a UX
  • it is shared with the team & feedback is gathered
  • @votdev can then implement it

Additional Information
@jhmarina jhmarina added kind/design Design work area/ui User Interface labels Oct 25, 2022
@jhmarina jhmarina added this to the v0.8.0 milestone Oct 25, 2022
@jhmarina
Copy link
Contributor Author

jhmarina commented Oct 25, 2022 •
edited

Testing wether the credentials are admin or not and then logging in is the ideal option, since it allows for removing the checkbox. We may not need a mockup.

@jhmarina jhmarina changed the title UI: UX to display wether user is admin or a system user on login UI: UX to display whether user is admin or a system user on login Oct 25, 2022
@jhmarina jhmarina changed the title UI: UX to display whether user is admin or a system user on login UI: UX for logging in as administrator Oct 25, 2022
@jhmarina jhmarina modified the milestones: v0.8.0, v0.9.0 Nov 14, 2022
@votdev
Copy link
Contributor

votdev commented Nov 16, 2022

Peek 2022-11-16 11-32

@votdev votdev mentioned this issue Nov 17, 2022
Merged
3 tasks
votdev referenced this issue in s3gw-tech/s3gw-ui Nov 17, 2022
@votdev
Merge pull request #96 from votdev/issue_175_admin_menu
f533ac0 
Fixes aquarist-labs/s3gw-tools#175
Fixes aquarist-labs/s3gw-tools#177
m-ildefons pushed a commit to m-ildefons/s3gw that referenced this issue Feb 6, 2023
@0xavi0
Merge pull request s3gw-tech#177 from 0xavi0/changelog-0.7.0
2578206 
Updated CHANGELOG.md with 0.7.0 release label
m-ildefons pushed a commit to m-ildefons/s3gw that referenced this issue Feb 6, 2023
@0xavi0
Revert "Merge pull request s3gw-tech#177 from 0xavi0/changelog-0.7.0"
c9f70ef 
This reverts commit 2578206, reversing
changes made to ad8bb8b.

Signed-off-by: Xavi Garcia <xavi.garcia@suse.com>
m-ildefons pushed a commit to m-ildefons/s3gw that referenced this issue Feb 6, 2023
@0xavi0
Merge pull request s3gw-tech#179 from 0xavi0/revert-0.7.0
14da26b 
Revert "Merge pull request s3gw-tech#177 from 0xavi0/changelog-0.7.0"
@jhmarina jhmarina mentioned this issue Mar 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@votdev votdev

Labels
area/ui User Interface kind/design Design work
Projects
None yet
Milestone
v0.9.0
Development

Successfully merging a pull request may close this issue.

UI: Combine the regular and admin user UI votdev/s3gw-ui
2 participants
@votdev @jhmarina