Skip to content

[19.0][FIX] upgrade_analysis: don't generate eval="''" attributes - #3541

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
hbrunn:19.0-upgrade_analysis-falsy-value-for-discarded-value-no-eval-empty-string
Mar 9, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
hbrunn:19.0-upgrade_analysis-falsy-value-for-discarded-value-no-eval-empty-string

Conversation

@hbrunn

@hbrunn hbrunn commented Mar 9, 2026

Copy link
Copy Markdown
Member

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @StefanRijnhart, @legalsylvain,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 19.0 milestone Mar 9, 2026
@hbrunn
hbrunn force-pushed the 19.0-upgrade_analysis-falsy-value-for-discarded-value-no-eval-empty-string branch from e8c811d to df965d6 Compare March 9, 2026 08:56
@hbrunn
hbrunn marked this pull request as ready for review March 9, 2026 08:56

@StefanRijnhart StefanRijnhart left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 19.0-ocabot-merge-pr-3541-by-StefanRijnhart-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 1b79993 into OCA:19.0 Mar 9, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

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.

5 participants