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

[17.0][MIG] hr_timesheet_analytic_tag: Migration to 17.0 #704

Merged
merged 7 commits into from
Oct 17, 2024

Conversation

BernatObrador
Copy link
Contributor

@BernatObrador BernatObrador mentioned this pull request Oct 15, 2024
18 tasks
@BernatObrador BernatObrador force-pushed the 17.0-mig-hr_timesheet_analytic_tag branch 2 times, most recently from ac65b38 to 9dc7f9e Compare October 15, 2024 07:49
@BernatObrador BernatObrador changed the title [MIG] hr_timesheet_analytic_tag: Migration to 17.0 [17.0][MIG] hr_timesheet_analytic_tag: Migration to 17.0 Oct 15, 2024
Copy link

@mpascuall mpascuall left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

Copy link
Contributor

@peluko00 peluko00 left a comment

Choose a reason for hiding this comment

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

LGTM, code review done

Copy link

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM, tested in runboat

@BernatObrador BernatObrador force-pushed the 17.0-mig-hr_timesheet_analytic_tag branch from 9dc7f9e to ec5b979 Compare October 17, 2024 05:44
@BernatObrador
Copy link
Contributor Author

@pedrobaeza Thanks for your contribution! The change has been made. Could you review it again, please?

@pedrobaeza
Copy link
Member

/ocabot migration hr_timesheet_analytic_tag

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Oct 17, 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 nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-704-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ca93d3b into OCA:17.0 Oct 17, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

8 participants