- Issue created by @eugen zerr
- @eugen-zerr opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 2:45pm 7 February 2023 - Status changed to Needs work
almost 2 years ago 3:26pm 14 February 2023 - 🇺🇸United States agentrickard Georgia (US)
This patch seems very specific to the stated configuration steps and I have concerns that it would break other configurations.
It definitely needs a test.
Questions that need to be answered:
- Why is the current user status relevant here?
- Can we even distinguish between "interface" and "content" configurations when the two are different?
As far as I understand, interface changes should be limited to string translations, not actual config differences that affect behavior of the application.