Re: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3

Поиск
Список
Период
Сортировка
От Khushboo Vashi
Тема Re: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3
Дата
Msg-id CAFOhELer_FxoVqaJ1oRU84fE1Tv=9Zu_b17qYMDCqn9Q500DJQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch] - RM 1402 - Macro support like pgAdmin3  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Список pgadmin-hackers
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 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.
           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. 
           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.
            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.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 Postgres
Mobile: +91 976-788-8246

Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Nikhil Mohite
Дата:
Сообщение: Re: [pgAdmin][RM3794]:Allow User to Change Database Connection from an Open Query Tool Tab
Следующее
От: Aditya Toshniwal
Дата:
Сообщение: [pgAdmin][RM4806] Add info to empty explain tab