Re: [pgAdmin4][Patch]: Fixed process watcher status message.

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: Fixed process watcher status message.
Date: 2018-10-25 11:16:48
Message-ID: CAM9w-_mEyKOZL1V9Jsn_BEu+daDEg2Ly3G+sYZpb_Yz+WJ9Z0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Akshay,

The patch looks good to me. But, I would suggest using some dictionary or
const variables for process states instead of hard code numbers 0,1,2,3. It
would be more readable and easier to change.

On Wed, Oct 24, 2018 at 5:48 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Hi All
>
> Please ignore my previous patch. Attached is the new modified patch.
>
> On Wed, Oct 24, 2018 at 3:03 PM Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch to show the correct message on the status bar of
>> the process watcher when user has stopped the process.
>>
>> --
>> *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*
>

--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2018-10-25 11:35:05 pgAdmin 4 commit: 1) Fixed process watcher status message when user has
Previous Message Akshay Joshi 2018-10-25 10:57:54 [pgAdmin4][Patch]: RM #3029 Cannot sort columns when importing data