You can add me to the patch. I might be able to get around to looking at it this week.
On Sat, Oct 15, 2016 at 11:47 AM, Tony Thomas 01tonythomas@gmail.com wrote:
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@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
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l