Re: [pgAdmin4][RM3849] Ask to save unsaved query changes stopped working
От | Dave Page |
---|---|
Тема | Re: [pgAdmin4][RM3849] Ask to save unsaved query changes stopped working |
Дата | |
Msg-id | CA+OCxoxbTp=niVEJown-P=Jk+m=AqETSUEkurdPYntXUg7CSpA@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin4][RM3849] Ask to save unsaved query changes stopped working (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>) |
Ответы |
Re: [pgAdmin4][RM3849] Ask to save unsaved query changes stopped working
(Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
|
Список | pgadmin-hackers |
On Tue, Jan 22, 2019 at 12:01 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote: > > > > On Tue, Jan 22, 2019 at 5:27 PM Dave Page <dave.page@enterprisedb.com> wrote: >> >> On Tue, Jan 22, 2019 at 11:54 AM Aditya Toshniwal >> <aditya.toshniwal@enterprisedb.com> wrote: >> > >> > Any suggestions on preferences name/desc ? >> >> Name: "Confirm on close" >> Desc: "Confirm closure of the browser or browser tab is intended >> before proceeding." > > It also warns on refresh. Name: "Confirm on close or refresh" Desc: "Confirm closure or refresh of the browser or browser tab is intended before proceeding." >> >> >> >> > On Tue, Jan 22, 2019 at 3:01 PM Dave Page <dave.page@enterprisedb.com> wrote: >> >> >> >> Hi >> >> >> >> On Tue, Jan 22, 2019 at 6:27 AM Akshay Joshi >> >> <akshay.joshi@enterprisedb.com> wrote: >> >> > >> >> > Hi Aditya >> >> > >> >> > Below are my review comments: >> >> > >> >> > You have added new config parameter "DISABLE_BEFOREUNLOAD_MESSAGE" to make this request configurable, should it bethe part of preferences setting instead of configuration? @Dave can you please comment on this. >> >> >> >> That does seem like something that should be a per-user preference, >> >> not a global config option. >> >> >> >> > Name "DISABLE_BEFOREUNLOAD_MESSAGE" is not clear as users perspective, can you please change it to some meaningfulname. >> >> > >> >> > Apart from that code looks good to me. >> >> > >> >> > On Mon, Jan 21, 2019 at 1:24 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote: >> >> >> >> >> >> Hi Hackers, >> >> >> >> >> >> Attached is the patch to throw alert warning when user reloads or closes the browser. >> >> >> >> >> >> Kindly review. >> >> >> >> >> >> -- >> >> >> Thanks and Regards, >> >> >> Aditya Toshniwal >> >> >> Software Engineer | EnterpriseDB Software Solutions | Pune >> >> >> "Don't Complain about Heat, Plant a tree" >> >> > >> >> > >> >> > >> >> > -- >> >> > Akshay Joshi >> >> > Sr. Software Architect >> >> > >> >> > >> >> > Phone: +91 20-3058-9517 >> >> > Mobile: +91 976-788-8246 >> >> >> >> >> >> >> >> -- >> >> Dave Page >> >> VP, Chief Architect, Tools & Installers >> >> EnterpriseDB: http://www.enterprisedb.com >> >> The Enterprise PostgreSQL Company >> >> >> >> Blog: http://pgsnake.blogspot.com >> >> Twitter: @pgsnake >> > >> > >> > >> > -- >> > Thanks and Regards, >> > Aditya Toshniwal >> > Software Engineer | EnterpriseDB Software Solutions | Pune >> > "Don't Complain about Heat, Plant a tree" >> >> >> >> -- >> Dave Page >> VP, Chief Architect, Tools & Installers >> EnterpriseDB: http://www.enterprisedb.com >> The Enterprise PostgreSQL Company >> >> Blog: http://pgsnake.blogspot.com >> Twitter: @pgsnake > > > > -- > Thanks and Regards, > Aditya Toshniwal > Software Engineer | EnterpriseDB Software Solutions | Pune > "Don't Complain about Heat, Plant a tree" -- Dave Page VP, Chief Architect, Tools & Installers EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company Blog: http://pgsnake.blogspot.com Twitter: @pgsnake
В списке pgadmin-hackers по дате отправления:
Предыдущее
От: Aditya ToshniwalДата:
Сообщение: Re: [pgAdmin4][RM3849] Ask to save unsaved query changes stopped working
Следующее
От: Akshay JoshiДата:
Сообщение: pgAdmin 4 commit: Proper SQL should be generated when create domainof