From: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
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 14:42:23 |
Message-ID: | CAFSMqn8GtZUbUoAiMigNKkTZS1uxqbZha9c_dgA9fwQ6d-YOEg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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/
Attachment | Content-Type | Size |
---|---|---|
sqleditorjs_refactor_v6.patch | text/x-patch | 14.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2019-08-23 15:00:13 | Re: [GSoC][New Feature] Editable and Read-only Columns |
Previous Message | Yosry Muhammad | 2019-08-23 14:16:58 | Re: [GSoC][New Feature] Editable and Read-only Columns |