Skip to content
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

Introduce embedded parameters for REST operations #6611

Draft
wants to merge 77 commits into
base: master
Choose a base branch
from

Conversation

lukedegruchy
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Jan 10, 2025

Formatting check succeeded!

…methods with return types, but still can't see them in Bruno. Introduce OperationEmbeddedParameter amd use it in place of OperationParameter. Try to figure out why caregaps doesn't use same setup code as evaluateMeasure.
…m. Add new params classes. Delete obsolete params classes. Add cloned operation params.
@lukedegruchy lukedegruchy changed the title WIP Introduce embedded parameters for REST operations Jan 15, 2025
…aseMethodBindingMethodParameterBuilder. Also, simplify OperationMethodBinding.
… Still need to fix Collection MethodUtil code as it doesn't work at request time.
… use ZonedDateTime. Some new testing and tweaks. javadoc.
…eant to replace the existing one and to use OperationParam. First steps.
…migrating OperationParameter and OperationParam to have parity with EmbeddedOperationParameter and EmbeddedOperationParam.
…migrate caregaps and evaluate measure(s) params. Still messy. Disable enforcer in pom.
…a. Redo javadoc and methods to remove those classes. Fix more logic to make tests pass. Disable enforcer plugin for good? Revive EmbeddedParameterConverter but don't completely phase out other code. Spotless.
…arameterConverter. Comment out enforcer plugin.
…n-operation parameter and non-embedded parameter annotations.
…thods from the methodutil branch. Branch out MethodUtil generics context into an outer class.
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