On Tue, Oct 24, 2017 at 6:08 PM, Alexander Lakhin <exclus...@gmail.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 <dp...@pgadmin.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: fix-get_locale_v2.patch
Description: Binary data

Reply via email to