Thread: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3
Hi,
Please find the attached patch for RM 1402 - Macro support like pgAdmin3.
Thanks,
Khushboo
Attachment
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.

- 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.
- 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.

- 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.

- 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.
On Mon, Sep 21, 2020 at 2:19 PM Khushboo Vashi <khushboo.vashi@enterprisedb.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 PostgresMobile: +91 976-788-8246
Attachment
Hi,
Please find the attached updated patch.
Thanks,
Khushboo
On Wed, Sep 23, 2020 at 5:42 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi KhushbooFollowing 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.
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.
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.
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.vashi@enterprisedb.com> wrote:Hi,Please find the attached patch for RM 1402 - Macro support like pgAdmin3.Thanks,Khushboo--Thanks & RegardsAkshay JoshipgAdmin Hacker | Sr. Software ArchitectEDB PostgresMobile: +91 976-788-8246
Attachment
Thanks, patch applied.
On Mon, Sep 28, 2020 at 2:18 PM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:
Hi,Please find the attached updated patch.Thanks,KhushbooOn Wed, Sep 23, 2020 at 5:42 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:Hi KhushbooFollowing 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.
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.
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.
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.vashi@enterprisedb.com> wrote:Hi,Please find the attached patch for RM 1402 - Macro support like pgAdmin3.Thanks,Khushboo--Thanks & RegardsAkshay JoshipgAdmin Hacker | Sr. Software ArchitectEDB PostgresMobile: +91 976-788-8246
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB PostgresMobile: +91 976-788-8246