Re: [pgAdmin4] [Patch]: Foreign Table Module

From: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
To: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4] [Patch]: Foreign Table Module
Date: 2016-04-05 08:57:05
Message-ID: CAFOhELekGKVKaMQt9L_itJw2WGdG69yVdiJDfq=1BPz3OjRJiw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Please find updated patch for the Foreign Table Module.

This patch is dependent on
1. Backgrid Depscell Patch, (submitted by me)
2. NodeAjaxOptionsCell Transform change patch, on which Ashesh and Murtuza
are working

Thanks,
Khushboo

On Wed, Feb 24, 2016 at 2:57 PM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi,
>
> I have updated the Foreign Table module as below:
>
> - Used 'NodeByListControl' to get schemas, in the foreign_table.js file as
> suggested by Ashesh to avoid code redundancy.
>
> - Applied *'Security Label Macro'* Patch (Implemented by Harshal).
> To test the Foreign Table patch, 'Security Label Macro' patch must be
> applied first as that is not committed yet.
>
> Please find attached Foreign Table Patch.
>
> Thanks,
> Khushboo
>
> On Tue, Feb 23, 2016 at 6:53 PM, Khushboo Vashi <
> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> Hi,
>>
>> Please find attached patch for the Foreign Table Module.
>>
>> The patch will be modified after Types module implementation as we need
>> to populate Base Type and some Type related validations from the Types
>> module.
>>
>> Please review it and let me know the feedback.
>>
>> Thanks,
>> Khushboo
>>
>
>

Attachment Content-Type Size
pgAdmin4_Foreign_tables_ver1.patch text/x-patch 102.6 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2016-04-05 09:20:10 [pgAdmin4] [Patch] Implementation of the Data Grid and Query Tool
Previous Message Ashesh Vashi 2016-04-05 08:00:56 Re: [pgAdmin4][psycopg2] - Modified status message implementation