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

Bugfix/missing xstream exception #214

Conversation

martink76
Copy link
Contributor

No description provided.

Copy link
Member

@smcvb smcvb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good, but I still have one request to make. We want to release this in 4.5.1 and for that, the PR should be merged into axon-kafka-4.5.x. If you could make that adjustment, @martink76, then we should be good to go.

Fix missed XStreamSerializer issue due to the Axon Framework 4.5.5 update
Fix XStreamSerializer issue due to the Axon Framework 4.5.5 update
@martink76 martink76 force-pushed the bugfix/missing-xstream-exception branch from d540a8f to 9437b22 Compare December 10, 2021 11:49
@martink76 martink76 changed the base branch from master to axon-kafka-4.5.x December 10, 2021 11:50
@martink76
Copy link
Contributor Author

@smcvb Thanks for the feedback. I've rebased onto axon-kafka-4.5.x.
Happy to hear that a 4.5.1 release is planned 👍

@lfgcampos lfgcampos requested a review from smcvb December 10, 2021 11:57
@martink76
Copy link
Contributor Author

Just to be sure - is there anything left I can do for this PR?

Copy link
Member

@smcvb smcvb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My concerns have been addressed, hence approving.

@smcvb
Copy link
Member

smcvb commented Dec 13, 2021

Just to be sure - is there anything left I can do for this PR?

Yep, that was getting my attention. 😅
My apologies for the wait; I think we're good to merge.

Thanks for the effort here, @martink76.
Any help is always very much appreciated! :-)

@smcvb smcvb merged commit f401fd9 into AxonFramework:axon-kafka-4.5.x Dec 13, 2021
@martink76
Copy link
Contributor Author

@smcvb @lfgcampos Thank you for the support 👍

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

Successfully merging this pull request may close these issues.

3 participants