Re: [patch][pgAdmin] RM3893 pgadmin4 Reassign/Drop Owned - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [patch][pgAdmin] RM3893 pgadmin4 Reassign/Drop Owned
Date
Msg-id CANxoLDc4KODrukZpCiRv7tFnVsbLartqRxRvp87C93K78M+0Mg@mail.gmail.com
Whole thread Raw
In response to [patch][pgAdmin] RM3893 pgadmin4 Reassign/Drop Owned  (Rahul Shirsat <rahul.shirsat@enterprisedb.com>)
Responses Re: [patch][pgAdmin] RM3893 pgadmin4 Reassign/Drop Owned  (Rahul Shirsat <rahul.shirsat@enterprisedb.com>)
List pgadmin-hackers
Hi Rahul

Following are the review comments:
  • The context menu should be renamed to 'Reassign/Drop Owned...' instead of 'Reassign/Drop role'.
  • Remove " role" from the dialog header. It should be "Reassign/Drop Owned - <role name>"
  • We have only two operations "Reassign and Drop", so instead of using Radio modern can we use switch control? Though radio modern also looks good. 
@Dave what do you think?
  • By default, Reassign operation should be selected.
  • After successfully reassign/drop we should show aleritfy notifier "Reassign owned successfully" or "Drop owned successfully".
  • "Roles dropped successfully!" string should be modified to "Drop owned successfully!".
  • Add validation if the database or reassign object is not selected, show them on GUI.
  • Connection to the database should be released(disconnected) after the execution of the query.
  • From Postgres 9.5 onwards we have "CURRENT_USER | SESSION_USER" please check that as well. From Postgres 14 CURRENT_ROLE is also there.
  • DROP OWNED query have [ CASCADE | RESTRICT ] which was not there even in pgadmin III, add support this would be good. Create two switch control for cascade and restrict and depending on that create the SQL query.
  • Documentation and screenshot update required after the above changes. The "Reassign/Drop Own" string should be replaced with "Reassign/Drop Owned" in the documentation.


On Mon, Jun 21, 2021 at 4:37 PM Rahul Shirsat <rahul.shirsat@enterprisedb.com> wrote:
Hi Hackers,

Please find the patch which includes the reassign/drop own feature.

This also includes documentation of the feature.

--
Rahul Shirsat
Senior Software Engineer | EnterpriseDB Corporation.


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: pgAdmin 4 - add missing gettexts and text space improvements
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed an issue where detaching the query editor panel