Fix Delete ContentItem Api

[hyzx86]

[sebastienros]

I don't think it's a solution to change the current behavior. I would prefer a new API that works on content item version ids, so you can delete a specific version, including drafts.

[jtkech]

Hmm, not sure but looks good to me that the API behavior is the same as the mvc controller, this by using VersionOptions.Latest meaning that an item is returned if any active version exists, and if so delete all active versions. In fact this is what we already do but only if a published version exists, in that case we already delete all active versions, if a draft also exists it is also deleted.

[Skrypt]

But, I think @sebastienros is right. Because you may want to remove a specific draft with a ContentItemVersionId instead. Else, this will only remove the latest ContentItem whether it is a draft or a published content item.

[jtkech]

Else, this will only remove the latest ContentItem whether it is a draft or a published content item.

No, it soft removes all active versions, published and latest if both exist

[sebastienros]

Ok with breaking changes but then only if we complete the API such that it's consistent and there is a way to do what was done before.

[Piedone]

Do you want to get back to this, @hyzx86?

[github-actions[bot]]

This pull request has merge conflicts. Please resolve those before requesting a review.

[hyzx86]

Close this PR for now, we should provide a new api to remove the specified version