From: | Nikhil Mohite <nikhil(dot)mohite(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][RM-6132]: [React] Port debugger to React |
Date: | 2022-06-14 12:38:53 |
Message-ID: | CAOBg0AP74Bz8KUnHE4R5vVGM-+TsyWJHRKyfGp0FLiu88-p0Hw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
PFA patch v2, fixed all review comments.
On Thu, Jun 9, 2022 at 4:53 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Nikhil
>
> Following are the review comments:
>
> *GUI:*
>
> - *Parameter dialog*:
> - Get the warning when parameter dialog gets opened Warning: [JSS]
> Could not find the referenced rule "disabled" in "makeStyles".
> - Debug button should not be enabled until all the required
> parameters value is not entered by the user. (Check the existing behavior)
> - OK, button not working on the 'Debugger Error' popup.
> - The 'Use Default' checkbox should be disabled if the default
> value is not available for the parameter. (Check existing behavior.)
> - Sometimes the value of the few parameters does not persist for
> the next run.
>
> Done.
>
> - *Debugger Panel*:
> - Fix 'Error while selecting frame.' error when clicking on the
> Line no of the 'stack' tab. Stop all the API calls when execution is
> completed.
> - Set Breakpoint not working. getting 'Error while clearing all
> breakpoint.' error on frontend and "UnboundLocalError: local variable
> 'line_numbers' referenced before assignment" error on the backend.
> - 'Debugger aborted successfully' message should be displayed in
> notistack when the user aborts the debugger execution. (Check the existing
> behavior)
>
> Done.
> *Code:*
>
> - Jasmine test cases are failing. Check the feature test as well.
> - Remove "debugger;" from the "direct.html" file.
> - Remove commented code from the "debugger/__init__.py" and "index.js"
> files. Also, remove the below else condition from
> "debugger/__init__.py" which is of no use
>
> else:
>
> []
>
>
> - Remove a few extra lines at the end of the "DebuggerModule.js".
> - Add copyright header to "DebuggerArgs.ui.js", "ToolBar.jsx" and
> "MockDebuggerComponent.jsx".
> - Fixed SonarQube issues from "DebuggerArgumentComponent.jsx",
> "DebuggerComponent.jsx".
>
> Done.
>
> On Wed, Jun 8, 2022 at 2:28 PM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Please find attached the patch for RM-6132
>> <https://redmine.postgresql.org/issues/6132>: [React] Port debugger to
>> React
>>
>>
>> --
>> *Thanks & Regards,*
>> *Nikhil Mohite*
>> *Senior Software Engineer.*
>> *EDB Postgres* <https://www.enterprisedb.com/>
>> *Mob.No: +91-7798364578.*
>>
>
>
> --
>
> <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/>
>
Regards,
Nikhil Mohite
Attachment | Content-Type | Size |
---|---|---|
RM-6132_v2.patch | application/octet-stream | 1.5 MB |
From | Date | Subject | |
---|---|---|---|
Next Message | Yogesh Mahajan | 2022-06-14 12:55:46 | Re: Feature #7178 - PostgreSQL deployment on Microsoft Azure |
Previous Message | Pradip Parkale | 2022-06-14 12:33:19 | Re: [pgAdmin][RM7351]: Templates not displayed |