Bug 39061 - Allow translation context in vue files
Summary: Allow translation context in vue files
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: I18N/L10N (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Martin Renvoize (ashimema)
URL:
Keywords:
Depends on:
Blocks: 38828 39045
  Show dependency treegraph
 
Reported: 2025-02-06 21:09 UTC by Caroline Cyr La Rose
Modified: 2025-04-23 07:42 UTC (History)
3 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:


Attachments
Bug 39061: Allow translation context in vue files (1.05 KB, patch)
2025-02-06 21:14 UTC, Caroline Cyr La Rose
Details | Diff | Splinter Review
Bug 39061: Export all functions from i18n.js (1.46 KB, patch)
2025-02-10 13:31 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 39061: Allow translation context in vue files (1.11 KB, patch)
2025-04-02 07:24 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review
Bug 39061: Export all functions from i18n.js (1.52 KB, patch)
2025-04-02 07:24 UTC, Martin Renvoize (ashimema)
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Caroline Cyr La Rose 2025-02-06 21:09:46 UTC
It is not currently possible to add translation context clues in vue files.
Comment 1 Caroline Cyr La Rose 2025-02-06 21:14:22 UTC
Created attachment 177600 [details] [review]
Bug 39061: Allow translation context in vue files

This patch adds the possibility to add translation context clues in vue
files.

Syntax is

$__p("context","term")

To test, see bug 39045.
Comment 2 Jonathan Druart 2025-02-10 13:17:10 UTC
I think we should provide all the functions defined in i18n.js here.
Comment 3 Jonathan Druart 2025-02-10 13:31:37 UTC
Created attachment 177693 [details] [review]
Bug 39061: Export all functions from i18n.js
Comment 4 Martin Renvoize (ashimema) 2025-04-02 07:24:13 UTC Comment hidden (obsolete)
Comment 5 Martin Renvoize (ashimema) 2025-04-02 07:24:19 UTC Comment hidden (obsolete)
Comment 6 Martin Renvoize (ashimema) 2025-04-02 07:26:10 UTC
Important, but clear and trivial, improvement.

Going straight for a QA here.. I'd love to see some basic unit tests, but we're well covered on the imported functions and this is just bringing them into the Vue space so I think we're OK without.

Passing QA
Comment 7 Katrin Fischer 2025-04-23 07:42:52 UTC
Pushed for 25.05!

Well done everyone, thank you!