From: | Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: User management functionality patch [pgadmin4] |
Date: | 2016-05-30 12:21:33 |
Message-ID: | CAM5-9D-+VZ2B-xqqMB5sfT1t7x7_3zPb0gs8DeEw1hdpvNQNEw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Harshal,
Please find the review comments so far:
*Issues:*
1. The UI design doesn't look interactive, especially add new user button.
2. In requirements.txt, while adding new dependent library the name of
library and version should be separate by '==' instead of '--'
3. Under File menu, "Users" menu item should not be visible for non-admin
users. For now it is only disabled.
4. Change role of admin user to non-admin user, then navigate to File >
Users. It doesn't show users in Dialog and Add user doesn't work. Instead
it is better to reload the page. Please find attached screenshot.
5. The data grid content should fill the unnecessary gap above the footer.
6. The title of delete user confirmation dialog should be "Delete user ?",
instead of "Delete User".
7. The Edit and Delete column shouldn't be sortable.
8. Columns 'creation time' & 'last modified' are missing.
9. Password should have validation of minimum number of characters.
*Can be added into TODO list:*
1. There is no way to search a user by name or email id.
2. Pagination should be there if there are large number of users.
3. There should be a way of notifying the user once its password is changed
by any administrator.
4. We can add 'created by' column.
5. One administrator shouldn't have privilege to modify the details of
other administrator.
This is the role of super admin.
Thanks
Surinder Kumar
On Fri, May 27, 2016 at 7:02 PM, Harshal Dhumal <
harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
> Hi,
>
> PFA initial patch for User management functionality.
>
>
> --
> *Harshal Dhumal*
> *Software Engineer *
>
>
>
> EenterpriseDB <http://www.enterprisedb.com>
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>
Attachment | Content-Type | Size |
---|---|---|
Screen Shot 2016-05-30 at 4.01.45 pm.png | image/png | 700.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Sandeep Thakkar | 2016-05-30 12:30:12 | Re: Patch for pgAdmin4 package on Mac OS X |
Previous Message | Ashesh Vashi | 2016-05-30 11:32:49 | Re: [pgAdmin4][Patch]: Refresh CONCURRENTLY is not supported on materialized view for pg-9.3 server |