Re: [pgAdmin][RM7031]: Database definition wrong - Mailing list pgadmin-hackers

From Pradip Parkale
Subject Re: [pgAdmin][RM7031]: Database definition wrong
Date
Msg-id CAJ9T6SsMhj=XhGmXSZp4Mi6Xcn6e80m8xTts-DBymooWPps+kw@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM7031]: Database definition wrong  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Responses Re: [pgAdmin][RM7031]: Database definition wrong
List pgadmin-hackers
Hi Akshay/Aditya,

Please find the updated patch.

On Wed, Dec 1, 2021 at 5:56 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Pradip,

You did not include the server_old table for query_history.
Fixed. 
Please also remove the commented code:
+    # db.engine.execute("PRAGMA foreign_keys=ON")
+
+    # db.session.commit()
+
+    # db.engine.execute("PRAGMA foreign_keys=ON")
+
Fixed. 
 


On Wed, Dec 1, 2021 at 5:25 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, the patch applied.

On Wed, Dec 1, 2021 at 4:23 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Hackers,

Please find the attached for #7031 Database definition wrong.

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


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



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


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

pgadmin-hackers by date:

Previous
From: Ashesh Vashi
Date:
Subject: Re: Patch: Two-factor Authentication (RM #6543)
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed some accessibility issues: