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
Fixes #23707: Implement common zio-json utils for plugins in rudder #5163
Fixes #23707: Implement common zio-json utils for plugins in rudder #5163
Conversation
.../sources/rudder/rudder-rest/src/main/scala/com/normation/rudder/rest/RudderJsonRequest.scala
Show resolved
Hide resolved
.../sources/rudder/rudder-rest/src/main/scala/com/normation/rudder/rest/RudderJsonRequest.scala
Outdated
Show resolved
Hide resolved
You need to add a description explaining the architecture/implem choice and - if needed - impact. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please:
- complete description (for ex "similar to RudderJsonResponse for the request side, nothing special")
- add missing headers
- avoid copy/pasting
PR updated with a new commit |
1 similar comment
PR updated with a new commit |
This PR is not mergeable to upper versions. |
OK, squash merging this PR |
69d67e3
to
1100c15
Compare
https://issues.rudder.io/issues/23707
It is about moving code from a plugin, because it needs to be shared by many plugins and also probably in the API endpoints definitions when migrating lift-json to zio-json...