https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2018-09-12T11:06:51ZCeph Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1204862018-09-12T11:06:51ZLenz Grimmer
<ul><li><strong>Tags</strong> changed from <i>dashboard</i> to <i>dashboard, usability, low-hanging-fruit</i></li></ul> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1207412018-09-14T09:22:50ZDan Guo
<ul></ul><p>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.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1207522018-09-14T13:09:17ZRicardo Marquesrimarques@suse.com
<ul></ul><p>dan Guo wrote:</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>IMO, we should also move the existing logout action to that popover (like in Github)</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1208382018-09-17T02:42:48ZDan Guo
<ul></ul><p>Ricardo Marques wrote:</p>
<blockquote>
<p>dan Guo wrote:</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>IMO, we should also move the existing logout action to that popover (like in Github)</p>
</blockquote>
<p>Yes, I think your idea is good, so I will implement a simple version about this popover and present a screenshot in the attachment.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1208512018-09-17T08:13:15ZLenz Grimmer
<ul></ul><p>dan Guo wrote:</p>
<blockquote>
<p>Yes, I think your idea is good, so I will implement a simple version about this popover and present a screenshot in the attachment.</p>
</blockquote>
<p>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.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1208572018-09-17T09:27:23ZDan Guo
<ul><li><strong>File</strong> <a href="/attachments/download/3713/user-name.png">user-name.png</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3713/user-name.png">View</a> added</li></ul><p>Lenz Grimmer wrote:</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>Hi, I already completed a simple version about this, while I use the drop-down menu instead of popover. Please see the attached.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1208592018-09-17T09:55:00ZLenz Grimmer
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/120859/diff?detail_id=119446">diff</a>)</li></ul> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1208682018-09-17T13:39:39ZVolker Theile
<ul></ul><p>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.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1209162018-09-18T07:22:38ZDan Guo
<ul><li><strong>File</strong> <a href="/attachments/download/3714/display-login-username.png">display-login-username.png</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3714/display-login-username.png">View</a> added</li></ul><p>dan Guo wrote:</p>
<blockquote>
<p>Hi, I already completed a simple version about this, while I use the drop-down menu instead of popover. Please see the attached.</p>
</blockquote>
<p>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.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1209182018-09-18T08:09:38ZLenz Grimmer
<ul></ul><p>dan Guo wrote:</p>
<blockquote>
<p>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.</p>
</blockquote>
<p><img src="https://tracker.ceph.com/attachments/download/3714/display-login-username.png" alt="" /></p>
<p>This looks great, thank you! Would you mind submitting a pull request for this?</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1209852018-09-19T01:57:51ZDan Guo
<ul></ul><p>Lenz Grimmer wrote:</p>
<blockquote>
<p>This looks great, thank you! Would you mind submitting a pull request for this?</p>
</blockquote>
<p>Of course I don't, I will create a new merge request on the master branch of ceph github repo.</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1209912018-09-19T06:08:39ZDan Guo
<ul></ul><p><a href="https://github.com/ceph/ceph/pull/24166" class="external">PR#24166</a></p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1210002018-09-19T07:20:57ZDan Guo
<ul></ul><p>dan Guo wrote:<br /><del><a href="https://github.com/ceph/ceph/pull/24166" class="external">PR#24166</a></del></p>
<p>Seems this pull request failed at Make Check in jenkins</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1210052018-09-19T08:54:02ZLenz Grimmer
<ul></ul><p>dan Guo wrote:</p>
<blockquote>
<p>dan Guo wrote:<br /><del><a href="https://github.com/ceph/ceph/pull/24166" class="external">PR#24166</a></del></p>
<p>Seems this pull request failed at Make Check in jenkins</p>
</blockquote>
<p>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".</p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1210112018-09-19T10:56:01ZRicardo Marquesrimarques@suse.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Fix Under Review</i></li></ul> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1210122018-09-19T10:56:35ZRicardo Marquesrimarques@suse.com
<ul></ul><p>PR: <del><a class="external" href="https://github.com/ceph/ceph/pull/24174">https://github.com/ceph/ceph/pull/24174</a></del></p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1211852018-09-24T08:06:07ZRicardo Marquesrimarques@suse.com
<ul></ul><p>PR: <a class="external" href="https://github.com/ceph/ceph/pull/24213">https://github.com/ceph/ceph/pull/24213</a></p> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1220532018-10-04T09:03:10ZRicardo Marquesrimarques@suse.com
<ul><li><strong>Status</strong> changed from <i>Fix Under Review</i> to <i>Resolved</i></li></ul> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1233622018-10-23T07:05:39ZLenz Grimmer
<ul><li><strong>Assignee</strong> set to <i>Dan Guo</i></li></ul> Dashboard - Feature #24822: mgr/dashboard: Display logged in userhttps://tracker.ceph.com/issues/24822?journal_id=1913852021-04-15T17:25:19ZErnesto Puerta
<ul><li><strong>Project</strong> changed from <i>mgr</i> to <i>Dashboard</i></li><li><strong>Category</strong> changed from <i>132</i> to <i>General</i></li></ul>