From: | Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin][RM4144] Compound Trigger support for EPAS-v12 |
Date: | 2019-08-08 09:49:53 |
Message-ID: | CAH4-4Wv9hNptwo6PvN_s7_LWacA8JE3Ts2c7UQdSOH34S5a-iQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Missed the attachment.
On Thu, Aug 8, 2019 at 2:59 PM Chethana Kumar <
chethana(dot)kumar(at)enterprisedb(dot)com> wrote:
> Hi Aditya,
>
> I am sending across the updated icons.
>
> ---
> Chethana kumar
>
> On Thu, Aug 8, 2019 at 12:19 PM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Akshay,
>>
>> Everything seems to be fine. I have a few suggestions:
>> 1) In class CompoundTriggerModule (python), "A module class for Trigger"
>> should be "A module class for Compound Trigger"
>> 2) We need to send a pull request to Code Mirror to highlight to make
>> show the word - COMPOUND as a keyword.
>> 3) Add the trigger name after END of the compound trigger definition. eg
>> END <cp trig>;
>> 4) Table SQL tab should show compound triggers.
>> 5) Compound trigger disabled icon is cut. @Chethana Kumar
>> <chethana(dot)kumar(at)enterprisedb(dot)com> - the icon need to be revisited.
>> [image: Screenshot 2019-08-08 at 11.58.27.png]
>> 6) In edit, SQL generated (msql) is having minor spacing issues on the
>> "FOR ..." line. Eg:
>> CREATE OR REPLACE TRIGGER comptrig
>> FOR INSERT OR DELETE OR UPDATE
>>
>> You can commit the code once fixed.
>>
>>
>> On Wed, Aug 7, 2019 at 4:00 PM Aditya Toshniwal <
>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>
>>> I'll do.
>>>
>>> On Wed, Aug 7, 2019 at 3:34 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>
>>>> Thanks Akshay. I want to do a pass over the docs before this is
>>>> committed. Otherwise, I don't see any obvious issues - however, it's a
>>>> pretty big patch, so can you have one of the others on the team do a
>>>> review/test as well please?
>>>>
>>>>
>>>> On Wed, Aug 7, 2019 at 8:08 AM Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hi Hackers
>>>>>
>>>>> I have added the support of *Compound Trigger* in pgAdmin4 for EPAS
>>>>> v12 and above. I have created a new collection node named "*Compound
>>>>> Triggers" *under the Tables node. API and RE-SQL test cases are
>>>>> covered and documentation is done.
>>>>>
>>>>> Please review the attached patch.
>>>>>
>>>>> --
>>>>> *Thanks & Regards*
>>>>> *Akshay Joshi*
>>>>>
>>>>> *Sr. Software Architect*
>>>>> *EnterpriseDB Software India Private Limited*
>>>>> *Mobile: +91 976-788-8246*
>>>>>
>>>>
>>>>
>>>> --
>>>> Dave Page
>>>> Blog: http://pgsnake.blogspot.com
>>>> Twitter: @pgsnake
>>>>
>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>> The Enterprise PostgreSQL Company
>>>>
>>>
>>>
>>> --
>>> Thanks and Regards,
>>> Aditya Toshniwal
>>> Software Engineer | EnterpriseDB India | Pune
>>> "Don't Complain about Heat, Plant a TREE"
>>>
>>
>>
>> --
>> Thanks and Regards,
>> Aditya Toshniwal
>> Software Engineer | EnterpriseDB India | Pune
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Chethana Kumar
> Principal UI/UX Designer
> EnterpriseDB Corporation
>
>
> The Postgres Database Company
>
> P: +91 86981 57146
> www.enterprisedb.com
>
--
Chethana Kumar
Principal UI/UX Designer
EnterpriseDB Corporation
The Postgres Database Company
P: +91 86981 57146
www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
compund_trigger.svg | image/svg+xml | 480 bytes |
compud_trigger_disable.svg | image/svg+xml | 706 bytes |
coll_compound_trigger.svg | image/svg+xml | 1.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Avin Kavish | 2019-08-08 10:32:47 | Food For Thought |
Previous Message | Chethana Kumar | 2019-08-08 09:29:51 | Re: [pgAdmin][RM4144] Compound Trigger support for EPAS-v12 |