Differential D8293
Use standard rendering and controller for Differential subscriptions Authored by epriestley on Feb 21 2014, 6:07 PM. Tags None Referenced Files
Subscribers
Details
Ref T2222. Differential has custom code for managing subscriptions, but no longer requires it. The one trick is that we don't have a hook for loading related data on the subscriptions workflow right now. Just glue that in for the moment; it's relatively harmless, and once Diffusion converts we'll have more context on how to best surface it properly. Subscribed and unsubscribed from a revision. Viewed different revisions and saw correct subscription state.
Diff Detail
Event TimelineComment Actions does this change where the action ends up in the list? no big deal to me, i would expect a complaint or two though if so. Comment Actions Yeah, it generally moves somewhere lower in the menu. I'd be surprised if we got too many complaints, but we'll see... |