Page MenuHomePhabricator

Change 'messages' to 'notices' internally (method names and such)
Open, NormalPublic

Description

Also, should we change the DB tables?

Event Timeline

Restricted Application removed a project: Patch-For-Review. · View Herald TranscriptJul 7 2016, 9:50 PM
Restricted Application added a subscriber: Zppix. · View Herald Transcript

Change 298953 had a related patch set uploaded (by Esanders):
Fix messages/notices documentation

https://gerrit.wikimedia.org/r/298953

Also class names and IDs you have a bit of a mess now with two different names, and the this.controller.getTypeString() === 'message' ? 'notice' : this.controller.getTypeString() hack

Change 298953 merged by jenkins-bot:
Fix messages/notices documentation

https://gerrit.wikimedia.org/r/298953