From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM3794]:Allow User to Change Database Connection from an Open Query Tool Tab |
Date: | 2020-09-21 11:54:06 |
Message-ID: | CANxoLDdUj_RKgQWFE6F7noEJbDz+P_3-njnn2mf10TSa0V9wdw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Nikhil
Following are the initial review comments:
- Open View/Edit data on any table and click on the same database
connection and then click on the Execute button. Got "get_primary_keys()
takes 1 positional argument but 2 were given" error.
- In my opinion, we should hide the option to change the database
connection for View/Edit Data.
- If the user clicks on the same database connection multiple times then
no need to change the backend connection and transaction id. Add validation
at the backend, no action required in this case.
- The role option is missing from the "connect to server" dialog.
- The Password field should not be there on the "connect to server"
dialog. Sometimes we saved the password so asking a password every time is
not correct. Check the pgAdmin 3 behavior.
Code review still remains.
On Thu, Sep 17, 2020 at 4:15 PM Nikhil Mohite <
nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
> Hi Team,
>
> Regarding RM-3794 <https://redmine.postgresql.org/issues/3794> allow
> the user to change the database connection from an open query tool:
> I have implemented the feature and also added documentation for it.
>
> PFA patch.
>
> --
> *Thanks & Regards,*
> *Nikhil Mohite*
> *Software Engineer.*
> *EDB Postgres* <https://www.enterprisedb.com/>
> *Mob.No: +91-7798364578.*
>
--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Sr. Software Architect*
*EDB Postgres <http://edbpostgres.com>*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2020-09-22 04:16:04 | pgAdmin 4 commit: Fixed an issue where the query tool is not fetching m |
Previous Message | Pradip Parkale | 2020-09-21 11:11:27 | Re: RM5841: server.username can be empty when using .pg_service.conf |