Reverted the commit. On Thu, Jan 21, 2021 at 3:13 PM Dave Page <dp...@pgadmin.org> wrote:
> This seems like a very bad idea. What if the user that has left was the > user that setup 50 connections used by everyone else? > > Deleting those shared entries is (I would guess) most likely *not* what > the majority of users would want, and the current behaviour is definitely > safest. > > We should make this optional; i.e. ask the use if they want shared servers > created by the user to be deleted. If they say no, they should be > reassigned to another user; either the admin that's deleting the user, or > their choice of user (a little more complex of course, but more flexible). > > Please revert this, until the deletion is made optional. > > On Thu, Jan 21, 2021 at 9:23 AM Akshay Joshi < > akshay.jo...@enterprisedb.com> wrote: > >> Thanks, patch applied. >> >> On Thu, Jan 21, 2021 at 12:18 PM Nikhil Mohite < >> nikhil.moh...@enterprisedb.com> wrote: >> >>> Hi Team, >>> >>> Please find the attached patch for RM-6143 >>> <https://redmine.postgresql.org/issues/6143>: Shared server entries not >>> getting deleted. >>> Added code to delete shared server entries if the admin deletes the user >>> from user management. >>> >>> >>> -- >>> *Thanks & Regards,* >>> *Nikhil Mohite* >>> *Software Engineer.* >>> *EDB Postgres* <https://www.enterprisedb.com/> >>> *Mob.No: +91-7798364578.* >>> >> >> >> -- >> *Thanks & Regards* >> *Akshay Joshi* >> *pgAdmin Hacker | Principal Software Architect* >> *EDB Postgres <http://edbpostgres.com>* >> >> *Mobile: +91 976-788-8246* >> > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EDB: http://www.enterprisedb.com > > -- *Thanks & Regards* *Akshay Joshi* *pgAdmin Hacker | Principal Software Architect* *EDB Postgres <http://edbpostgres.com>* *Mobile: +91 976-788-8246*