Skip to content

[17.0][OU-ADD] utm: Nothing to do #4664

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
Merged
OCA-git-bot merged 1 commit into from
Dec 10, 2024
Merged

[17.0][OU-ADD] utm: Nothing to do #4664

OCA-git-bot merged 1 commit into from
Dec 10, 2024

Conversation

@victoralmau
Copy link
Member

@victoralmau victoralmau commented Nov 14, 2024 •
edited
Loading

Nothing to do

@Tecnativa TT49337

@victoralmau victoralmau mentioned this pull request Nov 14, 2024
Merged
1 task
@legalsylvain
Copy link
Contributor

legalsylvain commented Nov 14, 2024

/ocabot migration utm

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Nov 14, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 14, 2024
Open
This was referenced Nov 14, 2024
Merged
Closed
Closed
Closed
@legalsylvain legalsylvain mentioned this pull request Nov 14, 2024
Closed
hbrunn
hbrunn requested changes Dec 2, 2024
View reviewed changes
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.

why do you do anything manually here? This should be taken care of by the ORM, so I'd file the module as nothing to do

@legalsylvain legalsylvain mentioned this pull request Dec 5, 2024
Closed
@hbrunn hbrunn mentioned this pull request Dec 9, 2024
Merged
@victoralmau victoralmau changed the title [17.0][OU-ADD] utm: Migration scripts [17.0][OU-ADD] utm: Nothing to do Dec 10, 2024
@legalsylvain
Copy link
Contributor

legalsylvain commented Dec 10, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Dec 10, 2024

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-4664-by-legalsylvain-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Dec 10, 2024

@legalsylvain The merge process could not be finalized, because command git push origin 17.0-ocabot-merge-pr-4664-by-legalsylvain-bump-nobump:17.0 failed with output:

To https://github.com/OCA/OpenUpgrade
 ! [remote rejected]         17.0-ocabot-merge-pr-4664-by-legalsylvain-bump-nobump -> 17.0 (cannot lock ref 'refs/heads/17.0': is at 81f47fd47e514fb632b12ca0f989754465edd3f8 but expected 72b56d3ade5818cd4dbc4fbe555e13b4a09de058)
error: failed to push some refs to 'https://github.com/OCA/OpenUpgrade'

@OCA-git-bot OCA-git-bot merged commit 81f47fd into OCA:17.0 Dec 10, 2024
2 checks passed
@OCA-git-bot
Copy link
Contributor

OCA-git-bot commented Dec 10, 2024

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

@pedrobaeza pedrobaeza deleted the 17.0-mig-utm branch December 10, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hbrunn hbrunn hbrunn approved these changes

Assignees

No one assigned

Labels

Dependency OK merged 🎉

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.

4 participants

@victoralmau @legalsylvain @OCA-git-bot @hbrunn