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.

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

[MIG] microsoft_outlook_single_tenant: Migration to 15.0 #1030

Merged
merged 6 commits into from
Jan 24, 2023
Merged

[MIG] microsoft_outlook_single_tenant: Migration to 15.0 #1030

merged 6 commits into from
Jan 24, 2023

Conversation

santostelmo
Copy link
Contributor

Migrating microsoft_outlook_single_tenant from 14.0 -> 15.0

@santostelmo santostelmo changed the title "[MIG] microsoft_outlook_single_tenant: Migration to 15.0" [MIG] microsoft_outlook_single_tenant: Migration to 15.0 Dec 19, 2022
@santostelmo santostelmo mentioned this pull request Dec 19, 2022
Open
36 tasks
@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). 🤖

@ajaniszewska-dev
Copy link

@pedrobaeza could we merge this one?

@pedrobaeza
Copy link
Member

The commits contents are incorrect, as there shouldn't be the version change in the black/isort commit. Please follow migration guide: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-15.0

@santostelmo
Copy link
Contributor Author

version change in the black/isort commit

I am not sure about what you mean about version change in the black/isort commit ?

@santostelmo santostelmo force-pushed the 15.0-mig-microsoft_outlook_single_tenant branch from c025550 to 83edd58 Compare January 23, 2023 08:26
@santostelmo
Copy link
Contributor Author

@pedrobaeza I changed the commit. Hope I interpreted well what you mean

@santostelmo santostelmo force-pushed the 15.0-mig-microsoft_outlook_single_tenant branch from 83edd58 to 46b4fc4 Compare January 23, 2023 09:27
@pedrobaeza
Copy link
Member

Yes, now the commits are correctly distributed, but there's a general error in the repository. @LoisRForgeFlow please fix mail_quoted_reply module tests.

@LoisRForgeFlow
Copy link
Contributor

@pedrobaeza The fix was already proposed by Stefan #1039 :)

@pedrobaeza
Copy link
Member

/ocabot migration microsoft_outlook_single_tenant
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jan 24, 2023
@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-1030-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d5e44cb into OCA:15.0 Jan 24, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 38947ae. 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
Reviewers

@leemannd leemannd leemannd approved these changes

@ajaniszewska-dev ajaniszewska-dev ajaniszewska-dev approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.
6 participants
@santostelmo @OCA-git-bot @ajaniszewska-dev @pedrobaeza @LoisRForgeFlow @leemannd