PIng again on this one, as we need review on
https://gerrit.wikimedia.org/r/#/c/304692, which is 2/3 of the shift to
ContentHandler patchsets.
Thanks to Legoktm, for reviewing the first one though. Its been a while
since the changes were posted (Aug 14), and super large
https://gerrit.wikimedia.org/r/#/c/295670/ was abandoned and split into
three:
[x]
https://gerrit.wikimedia.org/r/#/c/303984
[ ]
https://gerrit.wikimedia.org/r/#/c/304692/ and
[ ]
https://gerrit.wikimedia.org/r/#/c/309849/
The tracking phab task is
https://phabricator.wikimedia.org/T138462, which
was a GSoC 2015 project (its been almost 1 year)!
Thanks,
Tony Thomas <https://www.mediawiki.org/wiki/User:01tonythomas>
Home <http://www.thomastony.me> | Blog <https://tttwrites.wordpress.com/> |
ThinkFOSS <http://www.thinkfoss.com>
On Sat, Jul 16, 2016 at 6:08 PM, Tony Thomas <01tonythomas(a)gmail.com> wrote:
Hello all,
We have a patch
https://gerrit.wikimedia.org/r/#/c/295670/, which had its
last review almost 28 days back, and is a major blocker for the deployment
of Newsletter extension in production. The shift is tracked at T138462[1].
The patch is bit lengthy, and enable the extension to use ContentHandler,
which help us use a lot of in-wiki features. I had pulled the change to the
labs wiki at
http://newsletter-test.wmflabs.org/, which is broken as of
now, tracked in T138686[2].
It would be great if you devs can take a look at both the labs instance
and the contenthandler change.
[1]
https://phabricator.wikimedia.org/T138462
[1]
https://phabricator.wikimedia.org/T138686
Thanks,
Tony Thomas <https://www.mediawiki.org/wiki/User:01tonythomas>
Home <http://www.thomastony.me> | Blog <http://blog.thomastony.me> |
ThinkFOSS <http://www.thinkfoss.com>