Re: Query Tool JS Minimal Refactoring - Mailing list pgadmin-hackers

From Dave Page
Subject Re: Query Tool JS Minimal Refactoring
Date
Msg-id CA+OCxowzTbg+uCByBFh7qdFkHqeZDQzZhX11goema96B8DP6Zg@mail.gmail.com
Whole thread Raw
In response to Re: Query Tool JS Minimal Refactoring  (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>)
Responses Re: Query Tool JS Minimal Refactoring  (Yosry Muhammad <yosrym93@gmail.com>)
List pgadmin-hackers
Hi

On Fri, Aug 23, 2019 at 11:28 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Yosry,

The v3 patch has changes like:
diff --git a/docs/en_US/release_notes_4_13.rst b/docs/en_US/release_notes_4_13.rst
index bc21dfeca..a0a3a38ef 100644
--- a/docs/en_US/release_notes_4_13.rst
+++ b/docs/en_US/release_notes_4_13.rst
@@ -22,5 +22,4 @@ Bug fixes
 | `Issue #2706 <https://redmine.postgresql.org/issues/2706>`_ -  Added ProjectSet icon for explain module.
 | `Issue #2828 <https://redmine.postgresql.org/issues/2828>`_ -  Added Gather Merge, Named Tuple Store Scan and Table Function Scan icon for explain module.
 | `Issue #4643 <https://redmine.postgresql.org/issues/4643>`_ -  Fix Truncate option deselect issue for compound triggers.
-| `Issue #4644 <https://redmine.postgresql.org/issues/4644>`_ -  Fix length and precision enable/disable issue when changing the data type for Domain node.
-| `Issue #4650 <https://redmine.postgresql.org/issues/4650>`_ -  Fix SQL tab issue for Views. It's a regression of compound triggers.
\ No newline at end of file
+| `Issue #4644 <https://redmine.postgresql.org/issues/4644>`_ -  Fix length and precision enable/disable issue when changing the data type for Domain node.
\ No newline at end of file
diff --git a/web/pgadmin/browser/server_groups/servers/databases/schemas/views/__init__.py b/web/pgadmin/browser/server_groups/servers/databases/schemas/views/__init__.py


Looks like some other patch got mixed up.
Anyway, I've removed these changes and have attached the updated patch (v4) ready for commit. Everything looks good to me.

Like the editable columns patch, this one no longer applies.

Can we have a rebased version please?

Thanks! 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: Re: [GSoC][New Feature] Editable and Read-only Columns
Next
From: Yosry Muhammad
Date:
Subject: Re: Query Tool JS Minimal Refactoring