From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
Cc: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Query Tool JS Minimal Refactoring |
Date: | 2019-08-23 15:15:57 |
Message-ID: | CA+OCxoyhRk4d_neFP8FWJsLhaAWYmJHJ_rVtYN2kO+piF9qwbQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, committed.
On Fri, Aug 23, 2019 at 3:42 PM Yosry Muhammad <yosrym93(at)gmail(dot)com> wrote:
> No it isn't. I just needed to update JS tests after the latest
> modification and merging with master. Please find an updated patch
> attached, sorry for the inconvenience.
>
> On Fri, Aug 23, 2019 at 3:47 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Thanks - I assume this is dependent on the updatable columns patch, as
>> the JS tests fail if I apply it to the current GIT head?
>>
>> On Fri, Aug 23, 2019 at 1:07 PM Yosry Muhammad <yosrym93(at)gmail(dot)com>
>> wrote:
>>
>>> Please find an updated patch attached.
>>>
>>> On Fri, Aug 23, 2019 at 1:23 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Hi
>>>>
>>>> On Fri, Aug 23, 2019 at 11:28 AM Aditya Toshniwal <
>>>> aditya(dot)toshniwal(at)enterprisedb(dot)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
>>>>
>>>
>>>
>>> --
>>> *Yosry Muhammad Yosry*
>>>
>>> Computer Engineering student,
>>> The Faculty of Engineering,
>>> Cairo University (2021).
>>> Class representative of CMP 2021.
>>> https://www.linkedin.com/in/yosrym93/
>>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>
> --
> *Yosry Muhammad Yosry*
>
> Computer Engineering student,
> The Faculty of Engineering,
> Cairo University (2021).
> Class representative of CMP 2021.
> https://www.linkedin.com/in/yosrym93/
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Yosry Muhammad | 2019-08-23 22:38:45 | Re: [GSoC][New Feature] Editable and Read-only Columns |
Previous Message | Dave Page | 2019-08-23 15:15:46 | pgAdmin 4 commit: Some refactoring of sqleditor.js. |