Fix to JS to Purge in place #29
Conversation
| "purge-failed": "Purge failed" | ||
| "purge-failed": "Cache Purge failed", | ||
| "tooltip-n-purge-ext": "Purge cache of the current page", | ||
| "accesskey-n-purge-ext": "~" |
There was a problem hiding this comment.
Why have -n- in these messages?
There was a problem hiding this comment.
Because that’s the appropriate message key for tooltip and quick access key. -n- is needed.
There was a problem hiding this comment.
The -n- in other messages means 'navigation' and is used for items e.g. in the sidebar. If any shorthand should be used, it'd be -ca-, but really there's no need outside of core.
There was a problem hiding this comment.
Hmm that’s weird I said n but added another comment right after mentioning -ca- then pushed a change making them ca.
I see in some extensions not used, but in others it is.
There was a problem hiding this comment.
ah no worries, -ca- is good :)
|
What is the blocker of this? Is the fact this is a breaking change? I'm looking forward to this. |
mw-notify()is now available by default.Per Breaking changes for 1.35 release