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(a)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(a)gmail.com>
wrote:
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(a)lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l