Re: [pgAdmin][RM7404] Port process watcher to React

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: [pgAdmin][RM7404] Port process watcher to React
Date: 2022-08-16 12:20:13
Message-ID: CAM9w-_kD6Dsq+7AhqUyj97LhhjqkZt2QjSpCrYTpUb83pRrqTw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Hackers,

Attached patch fixes following issues:
1. Stop process of Process details dialog not working.
2. When a server of an in progress cloud deployment is expanded, it is
throwing a failed error.
3. Close the process notification on clicking View Processes.

Please review.

On Thu, Aug 11, 2022 at 10:54 AM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:

> Thanks, the patch applied.
>
> On Thu, Aug 11, 2022 at 9:52 AM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached is the updated patch. This includes GUI test cases failure fixes.
>>
>> On Tue, Aug 9, 2022 at 6:00 PM Aditya Toshniwal <
>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>
>>> Hi Hackers,
>>>
>>> Please ignore the previous patch. Attached the updated patch.
>>>
>>> On Fri, Aug 5, 2022 at 11:46 AM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> This patch essentially implements #3709 along with moving the process
>>>> watcher to React. The patch includes:
>>>> 1. Add a new tab "Processes" to view and manage all the background
>>>> processes.
>>>> 2. Processes tab allows to stop processes, remove process info and
>>>> logs, and view process logs and other details.
>>>> 3. The process watcher is removed. Instead, it will just notify if a
>>>> process is started, terminated, failed or completed. It will have a button
>>>> "View Processes" which will take you to the processes tab.
>>>> 4. New page of processes is added to the documentation and other pages
>>>> updated.
>>>> 5. JS Test cases added.
>>>>
>>>> Please review.
>>>>
>>>> --
>>>> Thanks,
>>>> Aditya Toshniwal
>>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>>> <http://edbpostgres.com>
>>>> "Don't Complain about Heat, Plant a TREE"
>>>>
>>>
>>>
>>> --
>>> Thanks,
>>> Aditya Toshniwal
>>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>>> <http://edbpostgres.com>
>>> "Don't Complain about Heat, Plant a TREE"
>>>
>>
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin Hacker | Software Architect | *edbpostgres.com*
>> <http://edbpostgres.com>
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
>
> <http://www.enterprisedb.com>
>
> Akshay Joshi
>
> Principal Software Architect
>
> +91 9767888246
>
> www.enterprisedb.com
>
> <https://www.linkedin.com/company/edbpostgres>
> <https://twitter.com/edbpostgres?lang=en>
> <https://www.facebook.com/EDBpostgres>
> <https://www.instagram.com/EDBpostgres/>
>

--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"

Attachment Content-Type Size
RM7404_v4.patch application/octet-stream 7.4 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2022-08-16 12:28:25 pgAdmin 4 commit: 1. Stop process of Process details dialog not working
Previous Message Akshay Joshi 2022-08-16 11:59:10 pgAdmin 4 commit: 1) Fixed an issue where an existing user is deleted a