- Assigned to ameymudras
- Issue was unassigned.
- Status changed to Needs review
8 months ago 2:13am 22 March 2024 - Status changed to Needs work
8 months ago 2:26am 22 March 2024 The Needs Review Queue Bot → tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- Status changed to Needs review
8 months ago 10:32am 22 March 2024 - 🇺🇸United States smustgrave
smustgrave → changed the visibility of the branch 11.x to hidden.
- Status changed to RTBC
8 months ago 3:48pm 24 March 2024 - 🇺🇸United States smustgrave
Removal of TranslationWrapper seems fine and didn't cause any issues
Changed the title to #32 as seems to more match what's happening here now.
- 🇬🇧United Kingdom longwave UK
I am not sure that this is a better solution than what we already have. The class alias will be set up on all sites, and we assume that only a very small number of sites actually need this - but we can't get rid of it entirely if it is serialised in unknown places - so isn't it better off left to the autoloader for those sites, but where it won't be loaded at all in the majority case?
- Status changed to Fixed
8 months ago 8:43am 25 March 2024 -
alexpott →
committed 67604e09 on 11.x
Issue #3269141 by ameymudras, longwave, ravi.shankar, sharayurajput,...
-
alexpott →
committed 67604e09 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.