Bug 37008 - "Help" link on ERM pages is not translatable
Summary: "Help" link on ERM pages is not translatable
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: ERM (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial
Assignee: Katrin Fischer
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-02 10:40 UTC by Katrin Fischer
Modified: 2024-12-06 07:48 UTC (History)
9 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.11.00,24.05.06
Circulation function:


Attachments
Bug 37008: Make "Help" link in vue modules translatable (1.58 KB, patch)
2024-06-27 16:48 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 37008: Make "Help" link in vue modules translatable (1.58 KB, patch)
2024-06-27 16:49 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 37008: Make "Help" link in vue modules translatable (1.65 KB, patch)
2024-07-01 14:38 UTC, Jake Deery
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2024-06-02 10:40:48 UTC
The "Help" link in the top right corner of the ERM module doesn't translate.
Comment 1 Katrin Fischer 2024-06-27 16:48:45 UTC
Created attachment 168207 [details] [review]
Bug 37008: Make "Help" link in vue modules translatable

The "Help" link in the upper right corner was not translatable
in the newer vue modules like ERM and Preservation.

To test in koha-testing-docker:

* Install any language:
  sudo koha-translate --install de-DE --dev kohadev
* Activate the language in the language system preference
* Activate the ERMModule system preference
* Go to the ERM module, verify translation works in general, but Help
  remains in English
* Apply patch
* Run: yarn build
* Run: gulp po:update
* Verify the string "Help" now appears in
  misc/translator/po/de-DE-messages-js.po
* Translate it
* Run:
  sudo koha-translate --update de-DE --dev kohadev
* Verify "Help" is now translated
Comment 2 Katrin Fischer 2024-06-27 16:49:22 UTC
Created attachment 168208 [details] [review]
Bug 37008: Make "Help" link in vue modules translatable

The "Help" link in the upper right corner was not translatable
in the newer vue modules like ERM and Preservation.

To test in koha-testing-docker:

* Install any language:
  sudo koha-translate --install de-DE --dev kohadev
* Activate the language in the language system preference
* Activate the ERMModule system preference
* Go to the ERM module, verify translation works in general, but Help
  remains in English
* Apply patch
* Run: yarn build
* Run: gulp po:update
* Verify the string "Help" now appears in
  misc/translator/po/de-DE-messages-js.po
* Translate it
* Run:
  sudo koha-translate --update de-DE --dev kohadev
* Verify "Help" is now translated
Comment 3 Pedro Amorim 2024-07-01 10:38:08 UTC
@Katrin should this be "Needs Sign-off"?
Comment 4 Katrin Fischer 2024-07-01 11:28:57 UTC
(In reply to Pedro Amorim from comment #3)
> @Katrin should this be "Needs Sign-off"?

It definitely should be, thanks Pedro.
Comment 5 Katrin Fischer 2024-07-01 11:29:33 UTC
Just for info: I am currently going through all the ERM templates, so more fixes coming soon (I need to do some more testing, where I was not sure)
Comment 6 Jake Deery 2024-07-01 14:38:45 UTC
Created attachment 168321 [details] [review]
Bug 37008: Make "Help" link in vue modules translatable

The "Help" link in the upper right corner was not translatable
in the newer vue modules like ERM and Preservation.

To test in koha-testing-docker:

* Install any language:
  sudo koha-translate --install de-DE --dev kohadev
* Activate the language in the language system preference
* Activate the ERMModule system preference
* Go to the ERM module, verify translation works in general, but Help
  remains in English
* Apply patch
* Run: yarn build
* Run: gulp po:update
* Verify the string "Help" now appears in
  misc/translator/po/de-DE-messages-js.po
* Translate it
* Run:
  sudo koha-translate --update de-DE --dev kohadev
* Verify "Help" is now translated

Signed-off-by: Jake Deery <jake.deery@ptfs-europe.com>
Comment 7 Katrin Fischer 2024-07-01 14:42:49 UTC
Thanks Jake!
Comment 8 Jake Deery 2024-07-01 14:45:16 UTC
Looks great to me, confirmed it as working following the test plan -- initially, added a translation string manually to misc/translator/po/es-ES-messages-js.po:

3617    #: koha-tmpl/intranet-tmpl/prog/js/vue/components/Help.vue:9
3618    msgid "Help"
3619    msgstr "Ayuda" <-- was just ""

I would suspect this was because the string isn't in Weblate, yet. Nonetheless, when I added Ayuda and switched to Spanish, I could see the string correctly translated.

Might want a second opinion on whether this is correct at the QA stage, but am happy to sign off as I feel the behaviour is accurate (I'm new to Koha dev, see!)

Signed off with thanks
Comment 9 Katrin Fischer 2024-07-09 12:50:11 UTC
Picked from QA queue for push to main.
Comment 10 Katrin Fischer 2024-07-09 12:56:44 UTC
Pushed for 24.11!

Well done everyone, thank you!
Comment 11 Lucas Gass (lukeg) 2024-11-18 17:53:27 UTC
Backported to 24.05.x for upcoming 24.05.06
Comment 12 Fridolin Somers 2024-12-06 07:48:21 UTC
Not for 23.11.x ?