Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@svandenhoek
Copy link
Contributor

@svandenhoek svandenhoek commented Oct 8, 2025

What are the main changes you did

  • Included /apps/{app}/ fix from feat: shacl validation dashboard #5291 to reduce future conflicts
  • Added OPTION request type support
  • <root>/api/rdf?shacls now validates on correct content-type
  • HEAD/GET now include Allow to communicate the allowed content-type's
  • Missing shaclSets now use NotFoundResponse to be coherent with how non-acceptable media types are processed (that is, not overriding ctx.status() but using the Javalin error so that it can be handled within MolgenisWebservice)

TODO

  • add tests

How to test

  • explain here what to do to test this (or point to unit tests)

Checklist

  • updated docs in case of new feature
  • added/updated tests
  • added/updated testplan to include a test for this fix, including ref to bug using # notation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants