Project

General

Profile

Actions

Feature #24822

closed

mgr/dashboard: Display logged in user

Added by Lenz Grimmer almost 6 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
General
Target version:
% Done:

0%

Source:
Tags:
dashboard, usability, low-hanging-fruit
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

PR#22758 has been merged and it's now possible to log in with multiple user accounts.

Currently, there is no visible indication of the logged in username. It should be possible to determine which user is currently logged in on the WebUI.


Files

user-name.png (1.45 KB) user-name.png Dan Guo, 09/17/2018 09:26 AM
display-login-username.png (3.7 KB) display-login-username.png Dan Guo, 09/18/2018 07:19 AM
Actions #1

Updated by Lenz Grimmer over 5 years ago

  • Tags changed from dashboard to dashboard, usability, low-hanging-fruit
Actions #2

Updated by Dan Guo over 5 years ago

In terms of the master branch of Ceph Git Repo has the new feature "user management" under the Setting drop-down menu on the top-right, we can found the login user name in the table of "users". But I found that if a user don't have all permissions of ceph, he/she would not see the "Setting" icon, and then he/she has not the enter to see the login user name. So, I want to do this job: add an icon on the top-right navigation list, and it is a popover, just show the current login user.

Actions #3

Updated by Ricardo Marques over 5 years ago

dan Guo wrote:

In terms of the master branch of Ceph Git Repo has the new feature "user management" under the Setting drop-down menu on the top-right, we can found the login user name in the table of "users". But I found that if a user don't have all permissions of ceph, he/she would not see the "Setting" icon, and then he/she has not the enter to see the login user name. So, I want to do this job: add an icon on the top-right navigation list, and it is a popover, just show the current login user.

IMO, we should also move the existing logout action to that popover (like in Github)

Actions #4

Updated by Dan Guo over 5 years ago

Ricardo Marques wrote:

dan Guo wrote:

In terms of the master branch of Ceph Git Repo has the new feature "user management" under the Setting drop-down menu on the top-right, we can found the login user name in the table of "users". But I found that if a user don't have all permissions of ceph, he/she would not see the "Setting" icon, and then he/she has not the enter to see the login user name. So, I want to do this job: add an icon on the top-right navigation list, and it is a popover, just show the current login user.

IMO, we should also move the existing logout action to that popover (like in Github)

Yes, I think your idea is good, so I will implement a simple version about this popover and present a screenshot in the attachment.

Actions #5

Updated by Lenz Grimmer over 5 years ago

dan Guo wrote:

Yes, I think your idea is good, so I will implement a simple version about this popover and present a screenshot in the attachment.

Sounds good - we're looking forward to your contribution! Thank you in advance. Let us know if you need any assistance - you can find us on the #ceph-dashboard IRC channel on OFTC.

Actions #6

Updated by Dan Guo over 5 years ago

Lenz Grimmer wrote:

Sounds good - we're looking forward to your contribution! Thank you in advance. Let us know if you need any assistance - you can find us on the #ceph-dashboard IRC channel on OFTC.

Hi, I already completed a simple version about this, while I use the drop-down menu instead of popover. Please see the attached.

Actions #7

Updated by Lenz Grimmer over 5 years ago

  • Description updated (diff)
Actions #8

Updated by Volker Theile over 5 years ago

If we introduce a new icon to the drop-down menu at the upper right corner then this menu should also provide some more useful entries like e.g. 'Settings' which directs the user to the user management 'Edit User' page.

Actions #9

Updated by Dan Guo over 5 years ago

dan Guo wrote:

Hi, I already completed a simple version about this, while I use the drop-down menu instead of popover. Please see the attached.

After the last implementation of 'show login username', I improved the style of the drop-down menu and gave a screenshot in the attachment. For this drop-down, we can add the "Settings" menu in it, just like in github.

Actions #10

Updated by Lenz Grimmer over 5 years ago

dan Guo wrote:

After the last implementation of 'show login username', I improved the style of the drop-down menu and gave a screenshot in the attachment. For this drop-down, we can add the "Settings" menu in it, just like in github.

This looks great, thank you! Would you mind submitting a pull request for this?

Actions #11

Updated by Dan Guo over 5 years ago

Lenz Grimmer wrote:

This looks great, thank you! Would you mind submitting a pull request for this?

Of course I don't, I will create a new merge request on the master branch of ceph github repo.

Actions #12

Updated by Dan Guo over 5 years ago

Actions #13

Updated by Dan Guo over 5 years ago

dan Guo wrote:
PR#24166

Seems this pull request failed at Make Check in jenkins

Actions #14

Updated by Lenz Grimmer over 5 years ago

dan Guo wrote:

dan Guo wrote:
PR#24166

Seems this pull request failed at Make Check in jenkins

Note that you don't have to close a pull request if you observe such a failure. Simply amend your commit and "push -f" it into your repo again. GitHub will update the pull request accordingly. But please consider using a dedicated branch, not "master".

Actions #15

Updated by Ricardo Marques over 5 years ago

  • Status changed from New to Fix Under Review
Actions #16

Updated by Ricardo Marques over 5 years ago

Actions #17

Updated by Ricardo Marques over 5 years ago

Actions #18

Updated by Ricardo Marques over 5 years ago

  • Status changed from Fix Under Review to Resolved
Actions #19

Updated by Lenz Grimmer over 5 years ago

  • Assignee set to Dan Guo
Actions #20

Updated by Ernesto Puerta about 3 years ago

  • Project changed from mgr to Dashboard
  • Category changed from 132 to General
Actions

Also available in: Atom PDF