We currently have support for monitoring indvidual artifacts and an entire app (i.e. all its artifacts implicitly). Let's now think how "Subscription" indicator (envelope) should work:
- If user is not subscribed to entire app, and not subscribed to ticket, the envelop should show "unsubscribed" and clicking should execute subscribe action.
- If user is not subscribed to entire app, but subscribed to ticket, the envelop should show "subscribed" and clicking should execute unsubscribe action.
- But if user subscribed to entire app, one cannot unsubscribe from just one artifact. The indicator should show some "implicitly subscribed" state and be unclickable.
Failing to capture these desticntions is partly cause of test failure in https://engr.geek.net/hudson/job/allura/358
I think 1 and 2 are right. I think for now if 3 is the case we need a different subscribe button (and a UI to allow subscribe to this ticket, or all tickets)
(2010-11-19 22:33:34) psokolovsky: Mark ramm-christensen: so, are you ok if I change button class (that's what test checks), and change it's clickability/tooltip for now?
(2010-11-19 22:33:44) Mark ramm-christensen: yes
(2010-11-19 22:33:47) Mark ramm-christensen: that is good
Reviewed by Wolf.