Re: [Wikitech-l] Need review on ContentHandler implementation, Newsletter extension

2016-10-18 Thread Tony Thomas
On Tue, Oct 18, 2016 at 4:30 AM, Aaron Schulz wrote: > You can add me to the patch. I might be able to get around to looking at > it this week. > Added you. Thanks a lot Aaron for helping in. Hope the extension reach production soon :) Thanks, Tony Thomas

Re: [Wikitech-l] Need review on ContentHandler implementation, Newsletter extension

2016-10-17 Thread Aaron Schulz
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 <01tonytho...@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 >

Re: [Wikitech-l] Need review on ContentHandler implementation, Newsletter extension

2016-10-15 Thread Tony Thomas
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