[12:05:14] <tonythomas>	 Glaisher: it would be great if you can review that one https://gerrit.wikimedia.org/r/#/c/295670 :)
[12:05:52] <Glaisher>	 tonythomas: I'm taking a quick look at it right now
[12:06:00] <tonythomas>	 Glaisher: Thank you :)
[12:13:57] <Glaisher>	 tonythomas: Added some comments. I only took a look at the code, not tested
[12:14:35] <tonythomas>	 Glaisher: thank you. the onSkinTemplateNavigation was directly copied with little modification from MassMessage :D
[12:14:45] <Glaisher>	 oh :)
[12:14:45] <tonythomas>	 just asked legoktm if we really need all those inside though
[12:15:02] <tonythomas>	 its one part of the patch which I got no clue about :D
[12:15:14] <tonythomas>	 will try removing them by part, and see how things break :)
[13:20:05] <tonythomas>	 Glaisher: its ready for another round of review ;)
[13:20:09] <tonythomas>	 https://gerrit.wikimedia.org/r/#/c/295670/
[16:28:18] <Glaisher>	 tonythomas: I just tested it. It breaks all the existing newsletters
[16:30:13] <Glaisher>	 Also I don't understand what the difference is supposed to be between the special page and the newsletter page now
[21:48:12] <tonythomas>	 Glaisher: break of all existing newsletters are expected as per legoktm
[21:48:39] <tonythomas>	 and also, I have to remove all those lines of code from  Special:Newsletter/id
[21:48:48] <tonythomas>	 as its no more useful.