From: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
---|---|
To: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Cc: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Subject: | Re: [pgAdmin][RM6130] React based framework for properties dialog and port Server Group, Server and Database dialogs |
Date: | 2021-06-28 09:20:36 |
Message-ID: | CAM9w-_=UjoitLEooheh5yF3vvZHdoSv4hMpD0z3Ge9g0ri1cwQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Hackers,
I made some changes to the core code that will be helpful for other nodes.
Please review.
On Thu, Jun 24, 2021 at 5:39 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Thanks, the patch applied in the "React_Porting" branch.
>
> On Mon, Jun 21, 2021 at 9:35 AM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Dave,
>>
>> On Thu, Jun 17, 2021 at 7:26 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Thu, Jun 17, 2021 at 11:01 AM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> Attached patch marks the beginning of migrating properties dialog/tab
>>>> to React based code, which is easy to maintain, performant and testable
>>>> using automation.
>>>> Patch includes:
>>>> - Framework for creating React based dynamic form view out of a
>>>> pre-defined UI schema. Previously, it was based on Backform/Backbone.
>>>> - The new framework and components will use MaterialUI as base.
>>>> Previously, Bootstrap/Backform/jQuery components were used.
>>>> - The new code uses JSS instead of CSS since material ui and most
>>>> modern React libraries also use JSS. In future, this will allow us to
>>>> change the theme in real-time without refresh.
>>>> - 90% code covered by 80-85 new jasmine test cases.
>>>> - Server group node UI Schema migration to new, with schema test cases.
>>>> - Server node UI Schema migration to new, with schema test cases.
>>>> - Database node UI Schema migration to new, with schema test cases.
>>>> - Few other UI changes.
>>>>
>>>
>>> Nice!
>>>
>>>
>>>>
>>>> PS: Until all the nodes are migrated, this will not go in the main
>>>> branch.
>>>>
>>>
>>> Yeah, how are we going to manage this? I agree with building it out in a
>>> branch until we have full coverage of the dialogues, but I'm concerned that
>>> it'll become a merge nightmare. Though, I guess we aren't touching the
>>> dialogues much for other things right now, so maybe not.
>>>
>> We'll right now work only on the dialogs. So merging should be manageable.
>> BTW, the patch can be merged in the main branch as well. It is backward
>> compatible, and it will work fine. Only the UI components would look
>> slightly different from the non-migrated dialogs.
>>
>>>
>>> --
>>> Dave Page
>>> Blog: https://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EDB: https://www.enterprisedb.com
>>>
>>>
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
>> <http://edbpostgres.com>
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
> *pgAdmin Hacker | Principal Software Architect*
> *EDB Postgres <http://edbpostgres.com>*
>
> *Mobile: +91 976-788-8246*
>
--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
<http://edbpostgres.com>
"Don't Complain about Heat, Plant a TREE"
Attachment | Content-Type | Size |
---|---|---|
RM6130.part2.patch | application/octet-stream | 11.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Aditya Toshniwal | 2021-06-28 10:27:10 | Re: [pgAdmin][RM6130] React based framework for properties dialog and port Server Group, Server and Database dialogs |
Previous Message | Dave Page | 2021-06-28 08:48:27 | pgAdmin 4 commit: Clarify when container config envvars may be ignored. |