Feature/axoniq/137/demo message transformation#4624
Open
laura-devriendt-lemon wants to merge 3 commits into
Open
Conversation
laura-devriendt-lemon
changed the base branch from
feature/axoniq/137/feature-message-transformator
to
enhancement/3597/feature-message-transformator
laura-devriendt-lemon
changed the base branch from
enhancement/3597/feature-message-transformator
to
feature/axoniq/137/feature-message-transformator
laura-devriendt-lemon
added
Type: Feature
| @@ -0,0 +1,112 @@ | |||
| # JacksonConverter + Jackson 2 inputs: why we chose fail-loud | |||
Contributor
Author
There was a problem hiding this comment.
discuss
laura-devriendt-lemon
requested review from
MateuszNaKodach,
hatzlj and
smcvb
and removed request for
a team
| * limitations under the License. | ||
| */ | ||
|
|
||
| package org.axonframework.examples.demo.coursecatalog.catalog.testutil; |
Contributor
Author
There was a problem hiding this comment.
should i make this part of the axoniq framework instead of demo? i know test fixtures where out of scope but i needed to make tests to verify it worked as expected
Contributor
Author
There was a problem hiding this comment.
if yes => reminder to self => adapt docs as well
hjohn
reviewed
Jun 4, 2026
hjohn
approved these changes
Jun 5, 2026
hjohn
left a comment
hjohn
left a comment
Contributor
There was a problem hiding this comment.
It is a lot, and I haven't seen all of it, but I did look at the examples. Now that we have samples for type safe variants, I'm pretty happy with how it looks.
laura-devriendt-lemon
force-pushed
the
feature/axoniq/137/demo-message-transformation
branch
from
03426ae to
6dad29a
Compare
The event transformation DSL moved from the EventTransformation class onto the sealed EventTransformer interface (axoniq-framework). Update the university-message-transformation demo to call EventTransformer.from(...). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The chain's register-after-build guard message changed from 'locked' to 'register() must be called before build()' in axoniq-framework #153; update the demo assertion to match. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add
university-message-transformationdemo moduleSits alongside
university-demoinexamples/. Demonstrates the AxonIQ Framework 5.2 event message transformation chain end to end. See the moduleREADME.mdfor setup, layout, and how to run.Spec coverage
All MUST-tier 5.2.0 FRs from the plan in https://github.com/AxonIQ/axoniq-framework/pull/153 are covered
Focused FR verification
Each FR should be backed by a named test the reviewer can run individually => for example
You can also choose to play around with the live shell and replaying by using AxonIQ Platform
Notes
.envis populated.