Copy alert types are untranslated

[Eva Cerninakova]

Copy alert types are not translated, see the attachment.
According to my finding, the string for copy alert types (like Checkin of lost copy, Checkin of missing copy, Checkin of lost-and-paid copy etc.) are completely missing in translation files in Launchpad.

[Linda Jansova (skolkova-s)]

It is still valid in Evergreen 3.3.2, making the copy alerts very difficult to use for staff members who are not good enough at English.

[Eva Cerninakova (ece)]

It is still valid in Evergreen 3.6.2

[Eva Cerninakova (ece)]

The problem remains unsolved in the Angular web staff client in the bug squashing test server (bugsquash2.mobiusconsortium.org), see the attachment.

[Bill Erickson (berick)]

Here's a fix:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1770979-copy-alert-type-xlate

From the commit:

Admin -> Local Admin -> Copy Alert Types is now an Angular page.

Copy Alert Type 'Name' field is now translatable

Fixes a formatting issue in Angular fm-editor so that the translate button appears on the same line as the value it means to translate.

Includes grid prefs SQL

[Eva Cerninakova (ece)]

Thanks Bill, that's great.

[Eva Cerninakova (ece)]

I have tested the bug using https://festivus.evergreencatalog.com/eg/staff/ server.

The copy alert type name seems to me definitely translatable, although, I wasn't able to test the translation fully because there is only English language available to translate to, which means, that adding English translation just changes the Copy Alert Type Name.

[Jane Sandberg (sandbergja)]

Thanks, Bill. I noticed a few regressions on the item alert type form:

* The order of the fields is different in the angular version from the angjs version. The fieldOrder should be: 'name,active,state,event,scope_org,next_status,in_renew,invert_location'
* The state field should be a dropdown with a list of status names
* The event field should be a dropdown with three options: Any Event / Checkin / Checkout
* The next status field should be a multi-select with a list of status names. It should be readonly unless the event field is set to Checkin.

[Galen Charlton (gmc)]

I've pushed an alternative patch to the branch user/gmcharlt/lp1770979_ccat_name_i18n

This focuses just on making the ccat.name field accessible to batch translation. As Jane noted, there's enough stuff going on in the AngularJS admin interface for ccat that it needs more than just the autogenerated Angular admin interface.

[Galen Charlton (gmc)]

I've also opened bug 2013792 regarding the Angularization of the Item Alert Types admin interface.