[go: up one dir, main page]

Skip to content

Make the endpoint for Intelligent Reviewer Assignment allow for PATCH / partial updates

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

[Part of Intelligent Reviewer Assignment]

This is an optional goal, but one that would be very useful for the front end, as it matches the UI that the MR page has evolved toward, with multiple reviewer assignment dropdowns that all show different subsets of the available list of reviewers.

Right now, the only way to modify the reviewers assigned to an MR is to replace the existing list of reviewers with a new list.

This worked well when there was a single dropdown that showed every eligible and selected reviewer, but it's now a liability with multiple segmented and filtered dropdowns.

It would be very convenient if we included a PATCH verb that could take a payload like: { "add": [ "X" ], "remove": [ "Z" ] } to do more piecemeal updates without fully replacing the reviewer list.

PATCH

This is slightly different than the intended purely RESTful way a PATCH would work, where the body would completely replace some portion of the target document, but I think "pure REST" is demonstrably not our goal at GitLab and this is in the correct spirit of a PATCH.

I'm also open to implementing deltas, operational transforms, and conflict-free replicated data types, but that might be... a tiny bit out of scope here.

Edited by 🤖 GitLab Bot 🤖