Fix a call to ngettext, and update to use the new plugin capabilities.

Review Request #11022 — Created May 15, 2020 and submitted

Information

Review Board
release-4.0.x

Reviewers

During the admin UI rewrite, I forgot a parameter to ngettext(). This
previously went unnoticed, but with our new babel-plugin-django-gettext,
it now becomes a build-time error.

This change fixes this up to provide the third argument, the count, and
at the same time switches to using the new syntax, simplifying the code
and reducing the setup for it.

Verified the generated code (when used against an upcoming build of
babel-plugin-django-gettext, which fixes ngettext-related issues).

Summary ID
Fix a call to ngettext, and update to use the new plugin capabilities.
During the admin UI rewrite, I forgot a parameter to `ngettext()`. This previously went unnoticed, but with our new babel-plugin-django-gettext, it now becomes a build-time error. This change fixes this up to provide the third argument, the count, and at the same time switches to using the new syntax, simplifying the code and reducing the setup for it.
eef32e97ae7f53997f2e85c481eef49a6eb1f938
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (aba0653)
Loading...