Code review comment for lp://staging/~gary/launchpad/bug777874

Revision history for this message
Gary Poster (gary) wrote :

As Robert said on IRC, reverting the union change will make qa easier. I'll do it.

For the js, since we don't do this generically (yet) I thought it would be surprising to update other tasks in response to saying "me too" because people might confuse coincidence with consequence. I'd be fine if we always kept statuses up-to-date with long-poll later, for instance; then we would not need this "something happened" code path at all. For now, though, I stand by my call there. Robert said on IRC he saw my point, and was fine with me proceeding as planned.

« Back to merge proposal