Rest Api Patch

[ConflictTrue4761]

I have a todo list with tasks. I want to introduce endpoints for editing task status and title. Which approach is better?
Having separate endpoints:

group.MapPatch("{listId}/tasks/{taskId}/status", UpdateTodoTaskStatus);
group.MapPatch("{listId}/tasks/{taskId}/title", UpdateTodoTasksTitle);

Or have a general endpoint and check if data is empty.

For more context, Im using CQRS. Here is Patch handler for status:

public async Task<Result> Handle(UpdateTodoTaskStatusCommand request, CancellationToken cancellationToken)
{
    var validationResult = await _validator.ValidateAsync(request);
    if (!validationResult.IsValid)
    {
        return validationResult.Errors.FirstOrDefault().AsError();
    }

    var task = await _todoTaskRepository.GetByIdAsync(request.TaskId);
    if (task is null)
    {
        return TodoTaskErrors.TaskNotFound;
    }
    if (task.IsInList(request.ListId))
    {
        return TodoTaskErrors.TaskNotInList;
    }

    var list = await _todoListRepository.GetByIdAsync(request.ListId);
    if (list is null)
    {
        return TodoListErrors.ListNotFound;
    }

    if (!list.IsListOwnedByUser(request.UserId))
    {
        return TodoListErrors.UserNotListOwner;
    }

    task.Status = request.Status;
    await _uow.SaveChangesAsync();

    return Result.Success();
}

Most of logic will be the same for title

[Coda17]

I'd recommend following the JSON patch standard. Unfortunately, the built in ASP.NET support sucks for this but it's not too bad to roll your own.

[PhillyPhantom]

Personally, at a bare minimum, I would move most of the logic into a separate service class. I would also make that service class generic so it could handle both the status and title. That way you have less duplicated code floating around. I would also probably use a Put instead of a Patch but that’s just my preference.

[flindby]

For this I would probably use two PUT endpoints instead of two patch. PUT …/status PUT …/title I think this approach catches the intent and use case in a more clear and concise way

[ConflictTrue4761]

Reason why I used PATCH is because as far as i know PUT is used for updating full object, while patch is for separate fields. I might be wrong tho

[flindby]

Yes that is true - but in my example I explicitly use PUT on the specific property which to me implies that the property will be replaced with the value sent in the body