- Issue created by @plopesc
- Merge request !7284Issue #3437576: Provide a new user.shortcut_set cache context β (Open) created by plopesc
- Status changed to Needs review
5 months ago 6:11am 3 April 2024 - πͺπΈSpain plopesc Valladolid
MR created to try to address this shortcut caching improvement.
- First commit to issue fork.
- Status changed to RTBC
5 months ago 1:32pm 5 April 2024 - πΊπΈUnited States smustgrave
Added some simple typehints nothing major.
I did purposely not add one to getLabel as the docs say it returns a string but translatable markup seems more correct. But definitely out of scope.
- Status changed to Needs work
5 months ago 9:00pm 5 April 2024 - π¬π§United Kingdom catch
I opened π Replace the shortcut_set_users table with a base field on users Active , if we had that, we wouldn't need the extra query, we could look up the shortcut set from the user object which we'll have anyway. I think this probably needs to be postponed on that issue.