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

[16.0][MIG] pos product display default code: migration to 16.0 #927

Merged
merged 5 commits into from
Feb 17, 2023
Merged

[16.0][MIG] pos product display default code: migration to 16.0 #927

merged 5 commits into from
Feb 17, 2023

Conversation

suker
Copy link
Contributor

@suker suker commented Feb 14, 2023

PR for migration from v15.0 to v16.0

related task: #849

@suker suker mentioned this pull request Feb 14, 2023
Open
37 tasks
@legalsylvain
Copy link
Contributor

/ocabot migration pos_product_display_default_code

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Feb 14, 2023
@suker suker changed the title [16.0][MIG] pos product display default code [16.0][MIG] pos product display default code: migration to 16.0 Feb 14, 2023
@suker suker force-pushed the 16.0-mig-pos_product_display_default_code branch 2 times, most recently from 2d12df4 to 6ece9eb Compare February 15, 2023 14:47
Pierrick Brun and others added 5 commits February 16, 2023 09:00
@suker suker force-pushed the 16.0-mig-pos_product_display_default_code branch from 6ece9eb to c05ac8d Compare February 16, 2023 08:01
chienandalu
chienandalu approved these changes Feb 16, 2023
View reviewed changes
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thanks for the improvements :)

@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-927-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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

To https://github.com/OCA/pos
 ! [remote rejected]   16.0-ocabot-merge-pr-927-by-ivantodorovich-bump-nobump -> 16.0 (cannot lock ref 'refs/heads/16.0': is at 756d9dac96597764a134b856323c371bf4e5e8ca but expected 38e244d22cb40d452c9de62fbbe557d2098e0255)
error: failed to push some refs to 'https://***@github.com/OCA/pos'

@OCA-git-bot OCA-git-bot merged commit 012eaac into OCA:16.0 Feb 17, 2023
@OCA-git-bot
Copy link
Contributor

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

@ivantodorovich ivantodorovich ivantodorovich left review comments

@chienandalu chienandalu chienandalu approved these changes

Assignees
No one assigned
Labels
merged 🎉
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
6 participants
@suker @legalsylvain @ivantodorovich @OCA-git-bot @chienandalu @manuelregidor