From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
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 06:07:51 |
Message-ID: | CANxoLDcWwwDhRXA-WupX1QuCKMwxBbNcC4S_=SpYjnLrOiRnGQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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.
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*
Attachment | Content-Type | Size |
---|---|---|
RM_3204_v3.patch | application/octet-stream | 11.7 KB |
![]() |
image/png | 49.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2018-05-22 07:09:00 | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |
Previous Message | Aditya Toshniwal | 2018-05-22 06:05:32 | [pgAdmin4][RM#3289] Can't query SQL_ASCII database. |