Skip to content

Conversation

@pedrobaeza
Copy link
Member

They were disconnected due to the column renaming, but can't be done before or we will get a FK constraint error.

@Tecnativa TT24299

@pedrobaeza pedrobaeza added this to the 12.0 milestone Jun 17, 2020
@pedrobaeza pedrobaeza changed the title [FIX] hr_holidays: Re-assign holidays to resource.calendar.leaves [12.0][FIX] hr_holidays: Re-assign holidays to resource.calendar.leaves Jun 17, 2020
@pedrobaeza
Copy link
Member Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot
Copy link
Contributor

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-2352-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

They were disconnected due to the column renaming, but can't be done before or we
will get a FK constraint error.
@pedrobaeza pedrobaeza force-pushed the 12.0-hr_holidays-rcl_holiday_id branch from c4bb5c5 to 7547a1a Compare June 17, 2020 16:53
@pedrobaeza
Copy link
Member Author

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 4573ba2 into OCA:12.0 Jun 17, 2020
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 12.0-hr_holidays-rcl_holiday_id branch November 18, 2020 16:13
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.

3 participants