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-18 11:26:45 |
Message-ID: | CA+OCxoxp5qdHY+T7=CHxSwgfBAmt81jpVGDHFmevW9Geoj-LhQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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.
--
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-18 12:28:04 | pgAdmin 4 commit: Major cleanup of the Windows installer build script. |
Previous Message | Akshay Joshi | 2018-05-18 11:11:28 | Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1 |