-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
story: 117304 implement openklant 2.0 #302
story: 117304 implement openklant 2.0 #302
Conversation
# Conflicts: # gradle/wrapper/gradle-wrapper.properties
# Conflicts: # gradle/wrapper/gradle-wrapper.properties
…on testing on macOS
# Conflicts: # gradle.properties
* fix integration tests not running on check/build * remove unnecessary Create* dto's
* expand filtering with page
* refactored some functions and class names
* introduce initial DigitaleAdres related entities and queries
* disable sql logging for app * fix DigitaleAdresType serialization * add authentication requirement to all openklant queries and mutations * add delete DigitaleAdres mutation
* change base integration test image
* integration tests for DigitaleAdres * fixed integration test ordering in test classe where it matters
Zou het geen idee zijn om als je een partij ophaalt dat je in de graphQL query ook aan kan geven of je via expand andere resources ingeladen kan krijgen en dan vooral ook welke resources je ingeladen wilt hebben. |
Dat had ik ook bedacht als een nice-to-have, alleen voor de huidige functionaliteit was het niet nodig. |
Merged for now to avoid getting a stale PR. Any desired changes should be done to the feature branch with a PR before we merge it to the development branch. |
This PR makes it possible to connect the NL Portal backend to an OpenKlant 2.X.X instance and interface with Partijen, PartijIdentificatoren and DigitaleAdressen and in the future other paths of the Klantinteracties API of OpenKlant 2.0.