Centralised notice extension (Homepage)
To make a confidential Fundraising task, visible only to members of acl*WMF-FR click here: https://phabricator.wikimedia.org/maniphest/task/create/?template=89899
Centralised notice extension (Homepage)
To make a confidential Fundraising task, visible only to members of acl*WMF-FR click here: https://phabricator.wikimedia.org/maniphest/task/create/?template=89899
There are also some suspicious code comments here ...
// For now, we want to make this display without paging on // meta.wikimedia.org, in line with the functionality that users currently // encounter. // This should be enough--Meta has less than 500 campaigns. private const DEFAULT_LIMIT = 5000;
meta.wikimedia.org probably has more than 500 campaigns now (every fundraiser year creates many) and why only care about Meta
Change #1057959 had a related patch set uploaded (by Abaris; author: Abaris):
[mediawiki/extensions/GrowthExperiments@master] Convert maybeShowWarning message boxes to Codex styles
Change #1057938 had a related patch set uploaded (by Abaris; author: Abaris):
[mediawiki/extensions/CentralNotice@master] Convert setValidationError message boxes to use Codex styles
The update for FlaggedRevs to use Codex message boxes has been completed as part of task T191156.
Change #1016759 abandoned by Jdlrobson:
[mediawiki/extensions/CentralNotice@master] Log central notice banner errors to their own channel
Reason:
See https://phabricator.wikimedia.org/T361680#9940111 for new proposal which requires modification to how the banners are added to the page.
@Pcoombe what do you think of the above proposal? Is it viable?
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Change #1052668 merged by jenkins-bot:
[mediawiki/extensions/CentralNotice@wmf_deploy] Merge branch 'master' into wmf_deploy
Sorry for the delay reviewing that merge! I've finished looking it over and smoke testing locally, and have given it C+2.
I shut down this VM due to lack of response here
In T366742#9981166, @Lucas_Werkmeister_WMDE wrote:Actually, it looks like this merge includes the fix, so IIUC that will roll out with this week’s train.
Actually, it looks like this merge includes the fix, so IIUC that will roll out with this week’s train.
This is still cluttering logspam-watch during deployments two months after it was fixed on the master branch. Can someone who knows how to deploy changes to CentralNotice please get the fix deployed?
No need for a RegEx. A little DOMDocument magic in BannerRenderer should do: