Page MenuHomePhabricator

Convert WatchstarApi to gateway class
Closed, ResolvedPublic

Description

We are converting our API classes to be gateways
(Preliminary reading: http://martinfowler.com/eaaCatalog/gateway.html)
This class should be converted to a gateway

Details

Related Gerrit Patches:
mediawiki/extensions/MobileFrontend : masterHygiene: Convert WatchstarApi to a gateway

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a subscriber: Jdlrobson.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 25 2015, 4:34 PM
Jdlrobson triaged this task as Low priority.Sep 25 2015, 4:34 PM
Jdlrobson set Security to None.

Change 238611 had a related patch set uploaded (by Jdlrobson):
WIP: Hygiene: Convert WatchstarApi to a gateway

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

Krenair added a subscriber: Krenair.

I'm pondering whether WatchstarGateway and WatchlistGateway should be merged given they overlap similar concepts... haven't done this in part of the patch. Thoughts?

I'm pondering whether WatchstarGateway and WatchlistGateway should be merged given they overlap similar concepts... haven't done this in part of the patch. Thoughts?

Sounds like a good idea, but can probably wait for another task/patch.

Change 238611 merged by jenkins-bot:
Hygiene: Convert WatchstarApi to a gateway

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

Jdlrobson closed this task as Resolved.Nov 2 2015, 7:01 PM