- last update
over 1 year ago 22 pass - 🇵🇹Portugal rutiolma
Actually, I was too fast on my previous comment since the LanguageInterface reference was missing.
Here's an updated patch.I'm also updating the version this patch applies to since 1.x is not supported anymore.
- First commit to issue fork.
- Merge request !33Issue #2912750: Expand multilingual support beyond text strings → (Open) created by mparker17
- Status changed to Needs work
8 months ago 12:39am 7 August 2024 - 🇨🇦Canada mparker17 UTC-4
I've ported the patch from comment #11 to a merge request; and made a slight improvement to the config schema. This will make it easier to review/merge in the future.
Overall, it looks fine: but one big thing preventing me from merging it is automated tests. I need automated tests, because I don't want future changes to the sitemap module to break this functionality, and break your website as a result! So, I'm moving this back to "Needs work".