From: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | "Libor M(dot)" <liborm85(at)gmail(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: pgAdmin 4 - next gettext usage fixes |
Date: | 2020-04-15 06:08:11 |
Message-ID: | CAM9w-_kyiCB-FFtkXpvUNYKsz2iauZcm-vDs30dgW7rXDdDD-A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Hackers/Libor,
The changes like below are incorrect. Try "Count rows" from a table's
context menu.
- info=gettext("Table rows counted: %s" % count),
+ info=gettext("Table rows counted: %s") % count,
Attached is the patch to fix all such changes in pgAdmin, to use format()
instead.
Please review.
On Fri, Apr 10, 2020 at 2:57 PM Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Hi Libor
>
> Thanks, patch applied. Please make sure to run the PEP8 checks before
> sending the patch.
> I have fixed and committed the code.
>
> On Wed, Apr 8, 2020 at 9:30 PM Libor M. <liborm85(at)gmail(dot)com> wrote:
>
>> Hello,
>> I fixed next gettext usage:
>>
>> - fixed gettext usage with .format() only for original text with %s
>> - fixed typos
>> - fixed translation yes/no buttons in dialog
>> - improved translating sentences without "connecting" words (eg. see
>> web/pgadmin/dashboard/static/js/dashboard.js, word 'cancel' needs to
>> be translated in Czech language as 'zrušit' but in another sentence as
>> 'zrušení')
>> - added gettext for text translations
>>
>> Diff file is attached.
>>
>> Best regards,
>>
>> Libor M.
>>
>> E-mail: liborm85(at)gmail(dot)com
>> GitHub: https://github.com/liborm85
>>
>
>
> --
> *Thanks & Regards*
> *Akshay Joshi*
>
> *Sr. Software Architect*
> *EnterpriseDB Software India Private Limited*
> *Mobile: +91 976-788-8246*
>
--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"
Attachment | Content-Type | Size |
---|---|---|
gettext_format_fixes.patch | application/octet-stream | 28.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amol Vyavahare | 2020-04-15 06:15:06 | [pgAdmin][RM4946] Issue with temporary tables on commit drop as. |
Previous Message | Khushboo Vashi | 2020-04-15 05:02:14 | [pgAdmin4][Patch]: RM 5366 - PgAdmin hangs in 'loading' state after trying to preview the first 100 records |