Skip to content

Fix OCM validation errors - #36667

Merged
provokateurin merged 1 commit into
masterfrom
fix/ocm-validation-errors
Mar 29, 2023
Merged

Fix OCM validation errors#36667
provokateurin merged 1 commit into
masterfrom
fix/ocm-validation-errors

Conversation

@provokateurin

Copy link
Copy Markdown
Member

Summary

According to the spec the validationErrors is an array of objects.

Checklist

@provokateurinprovokateurin mentioned this pull request Feb 10, 2023
8 tasks
@provokateurin
provokateurinforce-pushed the fix/ocm-validation-errors branch from 1ac359d to 0e62028CompareFebruary 10, 2023 21:55
Signed-off-by: jld3103 <jld3103yt@gmail.com>
@provokateurin
provokateurinforce-pushed the fix/ocm-validation-errors branch from 0e62028 to b271096CompareFebruary 10, 2023 21:56
@szaimenszaimen added this to the Nextcloud 26 milestone Feb 11, 2023
@szaimenszaimen added the 3. to review Waiting for reviews label Feb 11, 2023
@skjnldsvskjnldsv mentioned this pull request Feb 23, 2023
@blizzzblizzz mentioned this pull request Mar 7, 2023
@blizzzblizzz modified the milestones: Nextcloud 26, Nextcloud 27Mar 7, 2023
@provokateurin

Copy link
Copy Markdown
MemberAuthor

Technically this should be backported to every version, since it's a non-compliant implementation of the spec, but so far nobody complained, so it's probably not a big deal anyway, right?

@provokateurin
provokateurin merged commit fc371fa into masterMar 29, 2023
@provokateurin
provokateurin deleted the fix/ocm-validation-errors branch March 29, 2023 18:47
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@provokateurin@nickvergessen@ChristophWurst@blizzz@szaimen