Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working inversion 2.1
От | Dave Page |
---|---|
Тема | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working inversion 2.1 |
Дата | |
Msg-id | CA+OCxoz1BfCjmni_ATx5-N6Ct4YpGXrubdLQH7brrX8=+PoXqw@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 (Akshay Joshi <akshay.joshi@enterprisedb.com>) |
Ответы |
Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working inversion 2.1
|
Список | pgadmin-hackers |
Hi
--
On Tue, May 22, 2018 at 7:07 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Hackers,As per suggestion by Dave, I have modified the logic and now notifications are popped up in alertify dialog(refer Notify_Messages.png) as and when received on that session where "LISTEN" is executed. Attached is the modified patch, please review it.To test this feature following steps need to perform:
- Apply the patch.
- Run pgAdmin4
- Connect to any database server and open query tool.
- Execute 'LISTEN foo;' command.
- Open another query tool window and execute 'NOTIFY foo'. (This is without payload).
- Execute 'select pg_notify('foo', 'Hello')' query (with payload).
- Go to the query tool window from where 'LISTEN' was executed and run any other query.
I think there was a small misunderstanding here - I was suggesting that each notification be displayed in an Alertify notification, e.g. using alertify.message('A notification of FOO was received with payload '1234'...')
On Mon, May 21, 2018 at 1:36 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote: Hi DaveOn Fri, May 18, 2018 at 4:56 PM, Dave Page <dpage@pgadmin.org> wrote:On Fri, May 18, 2018 at 12:11 PM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote: Hi DaveOn Fri, May 18, 2018 at 3:58 PM, Dave Page <dpage@pgadmin.org> wrote:HiOn Wed, May 16, 2018 at 2:51 PM, Anthony Emengo <aemengo@pivotal.io> wrote:Hey,The code looks great! The tests all passed as well.Agreed - however, unless you check the Messages panel, you're not likely to see that a message was received.Can we also show each message in an alertify panel?We need to change the design I guess, as we are currently send this as part of Messages. We will have to send this separately and show it in the alertify panel.Yeah. Unfortunately I think notifications need to be more "active" than the messages.I am working on the above. Can we add one preferences setting to "ON/OFF" this alertify panel ?--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company--Akshay JoshiSr. Software ArchitectPhone: +91 20-3058-9517
Mobile: +91 976-788-8246--Akshay JoshiSr. Software ArchitectPhone: +91 20-3058-9517
Mobile: +91 976-788-8246
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления: