]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blob - doc/todo/_Add_ikiwiki_PO-Plugin_to_ignore_languages.mdwn
Gentle ping on proposed branch
[git.ikiwiki.info.git] / doc / todo / _Add_ikiwiki_PO-Plugin_to_ignore_languages.mdwn
1 [[!template  id=gitbranch branch=feature/15355-po-plugin-disable-languages author="[[intrigeri]]"]]
3 `po_disabled_languages` is used to disable languages on the website. It
4 can be used to prepare new translations, while it will hide those from
5 the end-user. More precisely, PO files for these languages are updated
6 normally, however, HTML creation will be disabled and no reference to
7 these translations will be visible on the website.
9 See also downstream bug report in tails: <https://labs.riseup.net/code/issues/15355>
11 > From the downstream bug report it appears this branch is not yet
12 > considered ready (as of February 2019), so I haven't attempted to
13 > review it.
14 >
15 > As with everything else in the `po` plugin: I would really want to see
16 > some regression test coverage before merging this, because as far as I'm
17 > aware none of the ikiwiki maintainers use the `po` plugin, so we are
18 > not able to detect regressions other than via the automated tests. That
19 > isn't a good situation to be in for a plugin that uses `IkiWiki::inject`
20 > to monkey-patch ikiwiki internals. If this plugin is important to you,
21 > more test coverage would be very much appreciated.
22 > --[[smcv]]
24 >> I've put this branch into a shape that I now dare submitting:
25 >> https://salsa.debian.org/tails-team/ikiwiki/tree/feature/15355-po-plugin-disable-languages.
26 >> It adds a little bit test coverage in the form of integration tests
27 >> for the proposed feature, that incidentally exercise the rest of the
28 >> basic functionality of the PO plugin.
29 >> Credits go to hefee for the initial work; I merely did a lot of polishing.
30 >> Happy to squash all this into a single commit if you prefer.
31 >> --[[intrigeri]]
33 >>> Please let me know if the added test coverage is sufficient to unblock
34 >>> the merge of this branch. Thanks in advance!
35 >>> --[[intrigeri]]
37 [[!tag patch]]