This would really improve the readability of the code...
See https://www.mediawiki.org/wiki/Manual:HTML_templates for documentation on how to use server-side HTML templates in MediaWiki.
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Use Mustache templates on the server side for result lines in CheckUser | mediawiki/extensions/CheckUser | master | +278 -240 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T139810 RFC: Overhaul the CheckUser extension | |||
| Open | • Niharika | T236225 [Epic] CheckUser 2.0 Improvements | |||
| Resolved | Dreamy_Jazz | T94735 Rewrite Special:CheckUser using server side templates |
Event Timeline
Comment Actions
@AronManning Can you provide more context on this ticket? I'm not sure what's being requested and why.
Comment Actions
@Niharika - This is just a code maintenance issue. No user-facing features would actually be affected by this (although it might make it easier to build such features).
Comment Actions
Change 814376 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):
[mediawiki/extensions/CheckUser@master] Use Mustache templates on the server side for result lines in CheckUser
Comment Actions
@Dreamy_Jazz have you identified a code reviewer for this one? Seems platform engineering are maintainers here so tagging accordingly.
Comment Actions
@Jdlrobson I've not identified a reviewer for this. Thanks for tagging the relevant team. I haven't been able to see if there is any tests for mustache templates, so if there is I would appreciate it being enabled on CheckUser if it isn't already.
Comment Actions
It's coming up to a month since I uploaded a patch. Is there anything I may have missed with this patch that needs to be addressed?
Comment Actions
Change 814376 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Use Mustache templates on the server side for result lines in CheckUser