- Issue created by @joachim
- First commit to issue fork.
- Merge request !9163issue/3385838: Add API() to lang setting for PathLanguageTest → (Closed) created by pooja_sharma
- Status changed to Needs review
3 months ago 9:54am 11 August 2024 Added API() to setup lang setting for PathLanguageTest & removed the http request, also user API() added to update user language instead of direct http request (for code optimise), apart from it nothing seems to be left.
Also verified test changes on local, its output on the basis of time & memory usage as follow:
Before applied MR changes test takes : Time: 00:09.965, Memory: 4.00 MB
After applied MR changes test takes: Time: 00:07.771, Memory: 4.00 MB
Please review, moving NR.
- Status changed to RTBC
3 months ago 6:39pm 14 August 2024 - 🇺🇸United States smustgrave
There's one change that may not be out of scope but may be fine.
- Status changed to Needs work
3 months ago 9:22pm 14 August 2024 - Status changed to Needs review
3 months ago 6:38am 15 August 2024 - Status changed to Needs work
3 months ago 10:44am 15 August 2024 - Status changed to Needs review
3 months ago 5:17am 16 August 2024 - Status changed to RTBC
3 months ago 12:24pm 20 August 2024 - 🇺🇸United States smustgrave
I believe feedback has been addressed on this one.
- Status changed to Fixed
3 months ago 9:27am 27 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.