Re: pgadmin4 l10n issues

From: Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>
To: Alexander Lakhin <exclusion(at)gmail(dot)com>
Cc: Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Subject: Re: pgadmin4 l10n issues
Date: 2017-10-24 14:17:31
Message-ID: CAG7mmoxNJHLVOhzWEw+-S5yK7ZXAQeviJJRgUeU09tjj=o=29Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Oct 24, 2017 at 6:08 PM, Alexander Lakhin <exclusion(at)gmail(dot)com>
wrote:

> Hi Ashesh,
>
> 23.10.2017 08:21, Ashesh Vashi wrote:
>
> Hi Dave/Alexander,
>
> On Thu, Oct 19, 2017 at 6:43 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Akshay, can you review/commit this, and look into why the remaining
>> strings Alexander notes are not getting translated please?
>>
>
>
> Did you forget to attach the patch?
>
> No, I didn't. You can find all my attached files here:
> https://www.postgresql.org/message-id/5c7914c8-db5f-cb70-
> a81e-6e9145039935%40gmail.com
>
I reviewed your patches.
I have committed the first patch.

About the second patch, I understood the issue.
I prefer another approach, instead of proposed by you.

Problem with the approach, you suggested, is: label for
'miscellaneous' Preference object will be always None.
Instead - read the 'raw_value' from the configuration database.

-- Thanks, Ashesh

>
>
> Best regards,
> ------
> Alexander Lakhin
> Postgres Professional: http://www.postgrespro.com
> The Russian Postgres Company
>
>

Attachment Content-Type Size
fix-get_locale_v2.patch application/octet-stream 2.3 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message pgAdmin 4 Jenkins 2017-10-24 14:37:18 Build failed in Jenkins: pgadmin4-master-python27 #360
Previous Message Ashesh Vashi 2017-10-24 13:22:20 pgAdmin 4 commit: Return the translations, and not empty array from the