Re: [pgAdmin4][Patch]: RM #3411 - pgAdmin 4, Mac. The application server could not be contacted

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM #3411 - pgAdmin 4, Mac. The application server could not be contacted
Date: 2018-06-14 07:50:07
Message-ID: CA+OCxoz0e8J6qGEzeGrQ6ViL17zpfZeRDFzudwP7A-Y1g14LzA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Thu, Jun 14, 2018 at 6:39 AM, Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

> Hi,
>
> Please find the fix for #3411 - pgAdmin 4, Mac. The application server
> could not be contacted
>
> The issue is with french translation which has been taken care in the
> attached patch.
>

We've seen similar issues with translations before. How can we write a test
(or tests) to validate them?

One idea that springs to mind is to generate a new template against the
app, then iterate through all the translation catalogs and attempt to
translate each string that we find in the template we generated.

Alternatively, just iterate the catalogs, and grab each of the source
strings from them and attempt to translate using the same catalog.

The former method would ensure that the current state of the application
can be translated, but that of course might miss some strings in the
catalogs. The latter would validate each catalog against itself, but might
miss some strings that are in the application but haven't been merged yet
(though, that doesn't seem like it would cause a problem).

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-06-14 08:27:04 pgAgent commit: Update file header comment.
Previous Message Dave Page 2018-06-14 07:44:18 Re: [pgadmin4][Patch]: Feature #2686 Property to switch lower/upper case auto completion