phenotips / remote-matching

Patient matching across applications
2 stars 2 forks source link

[misc] Fixed duplicate saving incoming requests in the final clause if request was successful #59

Closed veronikaslc closed 6 years ago

veronikaslc commented 6 years ago

@allasm comments addressed

allasm commented 6 years ago

Why did RM-75 ended up here not in RM-75 branch? Update: actually seem to be the right commit but with a wrong commit mesage. Maybe just squash the two, as they are kind of mutually exclusive

veronikaslc commented 6 years ago

@allasm commits squashed

allasm commented 6 years ago

Code approved, to be merged after testing

allasm commented 6 years ago

Closing this in favour of tested commits in the stable-1.1.x branch that have to be cherry-picked in the correct order into master