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

[18.0][MIG] purchase_analytic_global: Migration to 18.0 #2558

Open
wants to merge 16 commits into
base: 18.0
Choose a base branch
from

Conversation

yankinmax
Copy link
Contributor

No description provided.

@yankinmax yankinmax force-pushed the 18.0-mig-purchase_analytic_global branch 2 times, most recently from 519f564 to 42a9923 Compare February 13, 2025 14:38
@yankinmax
Copy link
Contributor Author

@ivantodorovich everything works completely fine, thanks for suggestions

Copy link
Contributor

@vvrossem vvrossem left a comment

Choose a reason for hiding this comment

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

Thank you for migrating this module 🙏

Copy link
Contributor

@vvrossem vvrossem left a comment

Choose a reason for hiding this comment

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

The icing on the 🍰

Squash administrative commits (if any) with the previous commit for reducing commit noise. Check https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests#mergesquash-the-commits-generated-by-bots-or-weblate for details.

🙏

@yankinmax
Copy link
Contributor Author

@vvrossem thanks for the review

@yankinmax yankinmax force-pushed the 18.0-mig-purchase_analytic_global branch from 42a9923 to 24b5c38 Compare February 13, 2025 18:22
@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). 🤖

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.

8 participants