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

[16.0][FIX] automation_oca: pass eval_context in domain #12

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

gaelTorrecillas
Copy link
Contributor

@gaelTorrecillas gaelTorrecillas commented Oct 7, 2024

Hello,
Following my first pr #8 , when we use a domain with today, datetime etc, we have an error when we add step.
So I add context in safe_eval contained in the step file.

Copy link
Member

@flotho flotho left a comment

Choose a reason for hiding this comment

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

IMHO, the message should be pass eval_context in domain

@gaelTorrecillas gaelTorrecillas changed the title [IMP] automation_oca: broadcast the domain in steps [FIX] automation_oca: pass eval_context in domain Oct 7, 2024
[FIX] automation_oca: pass eval_context in step domain safe_eval
Copy link
Member

@flotho flotho left a comment

Choose a reason for hiding this comment

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

LGTM

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 7, 2024
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-12-by-pedrobaeza-bump-patch, awaiting test results.

@pedrobaeza pedrobaeza changed the title [FIX] automation_oca: pass eval_context in domain [16.0][FIX] automation_oca: pass eval_context in domain Oct 7, 2024
@OCA-git-bot OCA-git-bot merged commit 0c6ccdf into OCA:16.0 Oct 7, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 06689b8. Thanks a lot for contributing to OCA. ❤️

@flotho
Copy link
Member

flotho commented Oct 7, 2024

thanks @pedrobaeza , we really appreciate your approbation.
Regards

@gaelTorrecillas
Copy link
Contributor Author

thanks you for yours review and approbation.

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.

4 participants