Re: [pgAdmin][RM7016]: Port Dependent, dependencies, statistics panel to React.

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin][RM7016]: Port Dependent, dependencies, statistics panel to React.
Дата
Msg-id CAM9w-_n2+Mn4o8Hm+i9J-FaXCjAA6VLDRTce=rDpjNfZ_9Z1-A@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM7016]: Port Dependent, dependencies, statistics panel to React.  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Ответы Re: [pgAdmin][RM7016]: Port Dependent, dependencies, statistics panel to React.  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
Список pgadmin-hackers

Hi Pradip,

We're moving towards React and not just replacing the UI HTML and keeping everything as it is. The current porting does not help when in future we remove wcDocker :
1.  'underscore', 'jquery', 'backbone', 'pgadmin.alertifyjs', 'pgadmin.backgrid' these should not be referred to at all after porting.

2. I can still see this code in the files:


// Defining Backbone Model for Dependencies.
var Model = Backbone.Model.extend({
  defaults: {
    icon: 'icon-unknown',

3. Since we're also removing jQuery - $.ajax should not be used.
$.ajax({
  url: url,
  type: 'GET',
})

4. There is no need for 2 files - dependencies.js and DependenciesComponent.jsx. 

There should be only one file - Dependencies.jsx.

5. There is no need to create Modules for dependencies. Dependencies should be directly mounted using (ReactDOM.render) from panel.js - handleVisibility

6. All other required logic should go inside - Dependencies.jsx. Just pass the required info to the Dependencies.jsx  component.

7. I have not checked the other two panels but I'm assuming that its done the same way.

7. The UI does not look good.
Screenshot 2022-01-05 at 1.54.31 PM.png
Screenshot 2022-01-05 at 1.26.31 PM.png

On Wed, Jan 5, 2022 at 9:30 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya

Can you please review it?

On Wed, 5 Jan, 2022, 9:18 am Pradip Parkale, <pradip.parkale@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached patch. I have a ported dependent , dependencies and statistics  panel to React.


--
Thanks & Regards,
Pradip Parkale
Software Engineer | EnterpriseDB Corporation


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Software Architect | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"
Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Akshay Joshi
Дата:
Сообщение: Re: [pgAdmin][RM7016]: Port Dependent, dependencies, statistics panel to React.
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Fixed review comments for Import/Export Servers.