Re: [pgAdmin4][Patch] Feature #3204 Notify/Listen not working in version 2.1

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-21 08:06:29
Message-ID: CANxoLDd1V6+NFQtBL55JYYWED0tdM2zzvdYHjOT4VWEm735CKQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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*

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-05-21 11:54:22 pgAdmin 4 commit: Don't barf on nested parentheses as we may find on 64
Previous Message Dave Page 2018-05-18 12:28:04 pgAdmin 4 commit: Major cleanup of the Windows installer build script.