Conversation
7895802 to
92eae9f
Compare
|
The quality gate problems were addressed in a separate PR |
1c44a2b to
04557f1
Compare
|
| @Path("/directory") | ||
| @Produces(MediaType.APPLICATION_JSON) | ||
| @Override | ||
| public Result importVoDs( |
There was a problem hiding this comment.
@burak-58 I understand that in principle we minimize changes to the rest API and keep things backward compatible, but in this particular case I see no way to keep this method here. The work is about to eliminate the concept of linked vod directories, so it has to be reflected in the API - any if we keep a backward compatible "ok"-ish response, that can cause more harm to customers than removing it. We need to increment the minor version in our semver to express the incompatibility and by removing the actual method, we make sure that it cannot go unnoticed on user end after an upgrade.
jantekb
left a comment
There was a problem hiding this comment.
left a comment on the removal of the rest api methods
Fix Sonar findings for VoD directory support
04557f1 to
1f24ac7
Compare
|



#7868