The message that is posted is "Status: rejected --> rejected" which isn't right. I think we need to keep the self.req.status = status line after the message posting, so the message has the right content.
if a project admin tries to update the merge request the regular way (e.g. set status to "merged"), nothing happens. I think the if statement in save( needs some more updates and testing of all the scenarios.
require_access will raise an exception not return None. has_access would be good to use instead.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Getting closer, but still a few things to adjust:
self.req.status = status
line after the message posting, so the message has the right content.if
statement insave(
needs some more updates and testing of all the scenarios.require_access
will raise an exception not return None.has_access
would be good to use instead.