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.

Sign up for GitHub

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

Jump to bottom

[OPIK-736] Fix deserialization issue #1022

Open

thiagohora wants to merge 2 commits into main from thiagohora/OPIK-736_fix_deserialization_issue

Contributor

thiagohora commented Jan 10, 2025

Details

The main problem was the AutomationRuleRowMapper, which pointed to AutomationRuleEvaluator instead of AutomationRuleEvaluatorModel.
There was also a tiny mistake related to nested transactions.
Finally, the mapping documentation was also fixed to so that the generation doesn't break.

I couldn't understand why this only happens within the docker container.

Issues #
OPIK-736

thiagohora marked this pull request as ready for review

January 10, 2025 18:32

thiagohora requested a review from a team as a code owner

January 10, 2025 18:32

thiagohora self-assigned this


          [OPIK-736] Fix deserialization issue

2b68503

thiagohora force-pushed the thiagohora/OPIK-736_fix_deserialization_issue branch from 7afbfde to 2b68503 Compare

January 10, 2025 18:34


          Merge branch 'main' into thiagohora/OPIK-736_fix_deserialization_issue

ff05e9e

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

Labels

None yet