Re: Regarding feature #642 - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: Regarding feature #642
Date
Msg-id CAM9w-_=CiAwW=4jJ5EeqbXDY==QA3neiXuM+9CNLPdLQwdBtRg@mail.gmail.com
Whole thread Raw
In response to Re: Regarding feature #642  (Nikhil Mohite <nikhil.mohite@enterprisedb.com>)
List pgadmin-hackers
Hi Nikhil,

It looks good to me. I have few suggestions here:
1. Can we remove the grey background for selected nodes?
2. Can/Should we add a label next to collections like - Tables (Selected: 3), d1 (Selected: 8), and when schema is selected then d1 (Selected: schema)?
3. We can have a three state checkbox for schema level checkbox. - empty, partial (select all child nodes) and full (schema selected).

Let me know what you think.

On Wed, Sep 13, 2023 at 11:06 AM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Hackers,

I have implemented feature #642 as follows.
1. If the user selects the schema node from the tree, it will use --schema to take a backup of that schema. (This is only when the user selects the schema node)
image.png
2. If a user selects a few nodes from the tree, like tables it will partially select the schema node and it will use the --table to take a backup of the selected nodes.
image.png
3. If the user selects all children of a schema but not the schema node then it will partially select the schema node and will use the --table to take a backup of the selected nodes.
image.png


Please let me know if you have any questions/ suggestions.


On Tue, Jul 4, 2023 at 10:30 AM Khushboo Vashi <khushboo.vashi@enterprisedb.com> wrote:


On Mon, Jul 3, 2023 at 6:16 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:
Hi Hackers,

I am currently working on #642 to allow to selection/deselect of objects under the object backup operation.

This will be only applicable to the database and its sub-nodes(Not for Backup Global and Backup server)

Details:
  1. Added a new tab in the backup dialog 'Objects'
  2. On the 'Objects' tab it will list down all the schemas and tables under the selected database.
  3. image.png
The second tab name (Data/Objects) should be changed because the last tab is named Objects.
  1. If the user selects the specific Schema then it will list down only tables under the selected schema.
  2. Screenshot 2023-07-03 at 4.38.26 PM copy.png
  3. By default, all the nodes from the tree will be selected and the user can deselect them if required.
+1 
  1. If the user de-selects all the nodes and took a backup it will take a normal backup and all other nodes will get included in the backup.
Then, selecting all the tree objects by default will make no sense. 
Questions:
  1. pgAdmin III use to support taking backup of tables only from the UI, a user was not able to select views, materialized views, foreign tables, and sequences from the backup dialog. (If the user selects all from the checkbox on the UI then other database objects are included implicitly in the backup) 
  2. Should we continue with the above behavior or should we now allow the user to select these database objects in the backup dialog?

Please let me know your suggestions/thoughts.


--
Thanks & Regards,
Nikhil Mohite
Senior Software Engineer.
Mob.No: +91-7798364578.

 Regards,
Nikhil Mohite


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Architect | enterprisedb.com
"Don't Complain about Heat, Plant a TREE"
Attachment

pgadmin-hackers by date:

Previous
From: Nikhil Mohite
Date:
Subject: [pgadmin-org/pgadmin4] aa973f: Added support to select/deselect objects in the Ba...
Next
From: Pravesh Sharma
Date:
Subject: [pgadmin-org/pgadmin4] d69b42: Fixed issues reported while testing feature #6378