From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
Cc: | Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: User management functionality patch [pgadmin4] |
Date: | 2016-05-30 10:47:18 |
Message-ID: | CAG7mmow740JeNcndqPeiEJ+eOOf6xJqi2RLqVXhJZCNSH7bP_A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, May 30, 2016 at 4:09 PM, Murtuza Zabuawala <
murtuza(dot)zabuawala(at)enterprisedb(dot)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(at)test(dot)com
> presnt and user try to add user same as test(at)test(dot)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
<http://www.enterprisedb.com/>
*http://www.linkedin.com/in/asheshvashi*
<http://www.linkedin.com/in/asheshvashi>
> 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(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
>>
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2016-05-30 11:32:14 | pgAdmin 4 commit: Support for the 'Refresh CONCURRENTLY' is applicable |
Previous Message | Murtuza Zabuawala | 2016-05-30 10:39:40 | Re: User management functionality patch [pgadmin4] |