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][MIG] outgoing_email_by_model: Migration from 15.0 to 16.0 #1512

Merged

Conversation

emiliesoutiras
Copy link
Contributor

No description provided.

@emiliesoutiras emiliesoutiras force-pushed the 16.0-mig-outgoing_email_by_model branch 3 times, most recently from 004c6d4 to c3f60a0 Compare November 27, 2024 16:41
Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

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

please follow the migration guide, most notably keeping commit history

@emiliesoutiras emiliesoutiras force-pushed the 16.0-mig-outgoing_email_by_model branch 2 times, most recently from 980547b to 17d0931 Compare December 11, 2024 18:34
Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

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

tests are fixed in #1519

@emiliesoutiras emiliesoutiras force-pushed the 16.0-mig-outgoing_email_by_model branch from a827cb6 to b1a4e0d Compare December 16, 2024 18:07
@emiliesoutiras
Copy link
Contributor Author

@Kev-Roche, @metaminux, to review

Copy link

@metaminux metaminux left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@hbrunn
Copy link
Member

hbrunn commented Dec 18, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-1512-by-hbrunn-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 569e51c into OCA:16.0 Dec 18, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1c4929c. 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.

7 participants