Page MenuHomePhabricator

CentralNotice: Stop adding old, empty bannerController modules to HTML
Closed, ResolvedPublic1 Estimated Story Points

Description

As part of last year's refactoring, we replaced ext.centralNotice.bannerController and ext.centralNotice.bannerController.mobile with ext.centralNotice.startUp. However, we never actually stopped adding the old modules to page HTML; instead, they were added as empty modules, with the new code as dependencies.

To simplify some minor code changes suggested by other teams, it would be useful to finally move forward with this simple change, and stop adding the old modules. One month after that (once all the Varnish cache clears) we should be able to fully remove their declarations from the code.

See Stage 2: New modules added to cached HTML.

Event Timeline

Change 296493 had a related patch set uploaded (by AndyRussG):
Request startUp and geoIP RL modules instead of bannerController

https://gerrit.wikimedia.org/r/296493

Change 296493 merged by jenkins-bot:
Request startUp and geoIP RL modules instead of bannerController

https://gerrit.wikimedia.org/r/296493