Issue #9 resolved

Support updating existing pullReviews with new information

Richard Warburton
created an issue

No description provided.

Comments (7)

  1. Mani Sarkar

    Found this in PullRequestImporter.java under method ensurePullReviewUpdated: // TODO - do we need to pass in a sensible linkToExternalInfo for the pull review modified event?

  2. Mani Sarkar

    Just finished implementing the TODO item, by applying the linkTOExternalInfo to modified pullReview (with necessary tests).

    Is that sufficient for this change? The above descriptions sort of expect more items to be populated. Will generate PR as soon as I get a confirmation.

    Daniel Bryant - I got to the bottom of the issue we were having earlier.

  3. Log in to comment