From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | Anthony Emengo <aemengo(at)pivotal(dot)io>, Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |
Date: | 2018-05-22 08:13:54 |
Message-ID: | CA+OCxoz1BfCjmni_ATx5-N6Ct4YpGXrubdLQH7brrX8=+PoXqw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Tue, May 22, 2018 at 7:07 AM, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)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(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi Dave
>>
>> On Fri, May 18, 2018 at 4:56 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>>
>>>
>>> On Fri, May 18, 2018 at 12:11 PM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Dave
>>>>
>>>> On Fri, May 18, 2018 at 3:58 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>
>>>>> Hi
>>>>>
>>>>> On Wed, May 16, 2018 at 2:51 PM, Anthony Emengo <aemengo(at)pivotal(dot)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 Joshi*
>>
>> *Sr. Software Architect *
>>
>>
>>
>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>
>
>
>
> --
> *Akshay Joshi*
>
> *Sr. Software Architect *
>
>
>
> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-05-22 08:32:52 | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |
Previous Message | Dave Page | 2018-05-22 08:07:56 | Re: [pgAdmin4][RM#3289] Can't query SQL_ASCII database. |