Last modified: 2014-09-08 20:03:19 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T68133, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66133 - Refactoring ZeroRatedMobileAccess into JsonConfig & ZeroBanner extensions
Refactoring ZeroRatedMobileAccess into JsonConfig & ZeroBanner extensions
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
Extension setup (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Yuri Astrakhan
:
Depends on:
Blocks: 31235
  Show dependency treegraph
 
Reported: 2014-06-04 19:31 UTC by Yuri Astrakhan
Modified: 2014-09-08 20:03 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Yuri Astrakhan 2014-06-04 19:31:16 UTC
ZeroRatedMobileAccess (ZRMA) extension is fairly complex, and requires substantial refactoring. I have moved its code into 3 extensions:

* JsonConfig - a general purpose extension to host JSON blobs as wiki pages (could  be reused by other extensions like EventLogger)

* ZeroBanner - code to show Zero banners for our zero carrier partners. Requires JsonConfig.

* ZeroPortal - this extension will be enabled on zero.wikimedia.org only and used for configuration management. Requires ZeroBanner.

The configuration patch: https://gerrit.wikimedia.org/r/#/c/136503/
Comment 1 Andre Klapper 2014-08-16 11:03:29 UTC
(In reply to Yuri Astrakhan from comment #0)
> The configuration patch: https://gerrit.wikimedia.org/r/#/c/136503/

...and that was merged.

What are the next steps here?
Comment 2 Andre Klapper 2014-09-07 19:25:09 UTC
(In reply to Yuri Astrakhan from comment #0)
https://gerrit.wikimedia.org/r/#/c/136503/

Yuri: That was merged. What are the next steps here?

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links