Details
-
Bug
-
Resolution: Timed out
-
Low
-
2.3.4
-
Severity 3 - Minor
-
Description
From http://forums.atlassian.com/thread.jspa?messageID=257348716#257348716
The last reviewer to complete generates an AllReviewersCompletedEvent, instead of a ReviewerCompletedEvent. The AllReviewersCompletedEvent does not contain a way of determining the originator of the event.
Two possible solutions:
1. The last reviewer to complete should generate both a ReviewerCompleteEvent and AllReviewersCompletedEvent. This is the most correct solution, but will cause disruption downstream for event consumers since they'll be getting two events instead of one for review completion. This does have the advantage of no API change though.
2. Expose the originator in the AllReviewersCompletedEvent. Not as disruptive, since the number of events generated will stay the same (although the API will change).