From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3 |
Date: | 2020-09-28 12:26:45 |
Message-ID: | CANxoLDdvDexWtGgcjO8JDGs9ZwsyD6tKDaWm9b9aO7H0OtZ2yg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, patch applied.
On Mon, Sep 28, 2020 at 2:18 PM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi,
>
> Please find the attached updated patch.
>
> Thanks,
> Khushboo
>
> On Wed, Sep 23, 2020 at 5:42 PM Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi Khushboo
>>
>> Following are the GUI review comments:
>>
>> - If we provide anyone value either Name or SQL it throws Alertify
>> error message with 'name' or 'SQL'. The error message should be correct.
>>
>> [image: Name_Error.png]
>>
> Fixed.
>
>>
>> - We should not enable the 'Save' button if the value of Name or SQL
>> is not entered by the user. Either both values should be there or both
>> should be blank.
>>
>> Fixed.
>
>>
>> - Click on anywhere inside the empty cell where the clear row icon is
>> not visible, it pops up the message to clear the row which is already
>> empty.
>>
>> [image: Clear_Row.png]
>>
> Fixed.
>
>>
>> - Save any macro and then open the "Manage Macros..." dialog again
>> click on the SQL control for the existing macro, cursor is not at the
>> appropriate place.
>>
>> [image: CodeMirror_Error.png]
>>
> Fixed.
>
>>
>> - Macros are not working for multiple users. Create some macro from
>> User1 and then log in using User2, macros created by User1 are visible to
>> User2 and when I try to update it is not updating.
>>
>> Fixed.
>
>>
>> On Mon, Sep 21, 2020 at 2:19 PM Khushboo Vashi <
>> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find the attached patch for RM 1402 - Macro support like pgAdmin3.
>>>
>>> Thanks,
>>> Khushboo
>>>
>>>
>>
>> --
>> *Thanks & Regards*
>> *Akshay Joshi*
>> *pgAdmin Hacker | Sr. Software Architect*
>> *EDB Postgres <http://edbpostgres.com>*
>>
>> *Mobile: +91 976-788-8246*
>>
>
--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Sr. Software Architect*
*EDB Postgres <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-09-28 12:26:59 | Re: [pgAdmin] Fix incorrect import in setup.py |
Previous Message | Akshay Joshi | 2020-09-28 12:25:12 | pgAdmin 4 commit: Added Macro support. Fixes #1402 |