Re: User management functionality patch [pgadmin4] - Mailing list pgadmin-hackers

From Ashesh Vashi
Subject Re: User management functionality patch [pgadmin4]
Date
Msg-id CAG7mmow740JeNcndqPeiEJ+eOOf6xJqi2RLqVXhJZCNSH7bP_A@mail.gmail.com
Whole thread Raw
In response to Re: User management functionality patch [pgadmin4]  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
List pgadmin-hackers

On Mon, May 30, 2016 at 4:09 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:

Hi Harshal,

PFA comments as below,

1) While running setup.py for the first time, We did not handle the case to create Standard user.
2) Save button gets enable only clicking on input text box
3) Remove delete button from sub-node
4) If current user changes its own name/email, main page should reflect new changes. 


Can be listed as TODO
--------------------------------
1) Erros description is missing, for example when have user test@test.com presnt and user try to add user same as test@test.com we get error as "Error during saving user" it should be descriptive like "ERRPR: User {XYZ} is already exists.."
2) There should be an option to delete multiple users at one shot.
Let's not do it right now.
You can add as TODO.

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


3) There should be an option to active/deactivate multiple users in one shot.
4) In-place editing in backgrid.


Regards,
Murtuza

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


On Fri, May 27, 2016 at 7:02 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi,

PFA initial patch for User management functionality.


-- 
Harshal Dhumal
Software Engineer 





--
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers



pgadmin-hackers by date:

Previous
From: Murtuza Zabuawala
Date:
Subject: Re: User management functionality patch [pgadmin4]
Next
From: Ashesh Vashi
Date:
Subject: pgAdmin 4 commit: Support for the 'Refresh CONCURRENTLY' is applicable