[03:26:53] addshore: Hi [09:46:46] tinajohnson__: hi! Just getting to the office [09:48:12] addshore: okay :). Could you look at this https://gerrit.wikimedia.org/r/#/c/267549/2 when you get some time ? [09:48:37] I've a patch for the same file and could avoid rebase issues if this one gets merged :) [09:58:29] tgindone [09:58:31] tinajohnson__: done [09:58:46] addshore: Thank you :) [09:59:53] addshore: Do you happen to know if each echo notification needs a new presentation model class or just one for all notifications? [10:00:08] tinajohnson__: no idea *goes to look* [10:00:57] tinajohnson__: looks like it needs a different one each time as you construct it with the Event and User [10:01:27] addshore: ah okay, I was confused about that