Re: Array representation and save array data from datagrid [RM2671][pgAdmin4]

From: Shirley Wang <swang(at)pivotal(dot)io>
To: Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Array representation and save array data from datagrid [RM2671][pgAdmin4]
Date: 2017-09-15 01:58:31
Message-ID: CAPG3WN5J=OLySQkb4Q49-roTsh92F1HZPuj656XaUv53hVBz7A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Harshal and Akshay,

I'm noticing that the screenshots you attached have fonts that look
different (ex. no font weights for table headers). Is that a result of the
font bundling and needs to be updated?

On Fri, Sep 15, 2017 at 12:18 AM Harshal Dhumal <
harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:

> Thanks, I'll look into it.
>
> On Sep 14, 2017 5:31 PM, "Akshay Joshi" <akshay(dot)joshi(at)enterprisedb(dot)com>
> wrote:
>
>> Hi Harshal
>>
>> I have reviewed your patch it looks good to me. Only one issue I have
>> found with char[], when I store {'a','b','c'} it gets stored and when I
>> refreshed it shows {',','}. Please look into this issue.
>>
>> On Thu, Sep 14, 2017 at 3:54 PM, Akshay Joshi <
>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>
>>> It works, sorry for the noise.
>>>
>>> On Thu, Sep 14, 2017 at 3:48 PM, Akshay Joshi <
>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>
>>>> Harshal,
>>>>
>>>> Can you please rebase the patch, it's not applied with the latest code.
>>>>
>>>> On Tue, Sep 12, 2017 at 2:37 PM, Akshay Joshi <
>>>> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>>>>
>>>>>
>>>>>
>>>>> On Tue, Sep 12, 2017 at 2:34 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>
>>>>>> Adding Akshay...
>>>>>>
>>>>>
>>>>> Sure.
>>>>>
>>>>>>
>>>>>> On Tue, Sep 12, 2017 at 10:04 AM, Dave Page <dpage(at)pgadmin(dot)org>
>>>>>> wrote:
>>>>>>
>>>>>>> Akshay, can you review/commit this please?
>>>>>>>
>>>>>>> On Mon, Sep 11, 2017 at 3:40 PM, Harshal Dhumal <
>>>>>>> harshal(dot)dhumal(at)enterprisedb(dot)com> wrote:
>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> Please find attached patch to represent array data in datagrid with
>>>>>>>> curly braces and
>>>>>>>> also allow user to save array with values like <null>, <empty
>>>>>>>> string>, '*""*' and "*''*"
>>>>>>>>
>>>>>>>> --
>>>>>>>> *Harshal Dhumal*
>>>>>>>> *Sr. Software Engineer*
>>>>>>>>
>>>>>>>> EnterpriseDB India: http://www.enterprisedb.com
>>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>
>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Dave Page
>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>> Twitter: @pgsnake
>>>>>>
>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>> The Enterprise PostgreSQL Company
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Akshay Joshi*
>>>>> *Principal Software Engineer *
>>>>>
>>>>>
>>>>>
>>>>> *Phone: +91 20-3058-9517 <+91%2020%203058%209517>Mobile: +91
>>>>> 976-788-8246 <+91%2097678%2088246>*
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> *Akshay Joshi*
>>>> *Principal Software Engineer *
>>>>
>>>>
>>>>
>>>> *Phone: +91 20-3058-9517 <+91%2020%203058%209517>Mobile: +91
>>>> 976-788-8246 <+91%2097678%2088246>*
>>>>
>>>
>>>
>>>
>>> --
>>> *Akshay Joshi*
>>> *Principal Software Engineer *
>>>
>>>
>>>
>>> *Phone: +91 20-3058-9517 <+91%2020%203058%209517>Mobile: +91
>>> 976-788-8246 <+91%2097678%2088246>*
>>>
>>
>>
>>
>> --
>> *Akshay Joshi*
>> *Principal Software Engineer *
>>
>>
>>
>> *Phone: +91 20-3058-9517 <+91%2020%203058%209517>Mobile: +91 976-788-8246
>> <+91%2097678%2088246>*
>>
>

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message git 2017-09-15 04:44:19 pgAdmin 4 commit: Branch refs/heads/WEBPACK_DEVEL was created
Previous Message Harshal Dhumal 2017-09-14 16:18:28 Re: Array representation and save array data from datagrid [RM2671][pgAdmin4]