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

[MIG] pos_receipt_hide_price: Migration to 16.0 #851

Merged
merged 10 commits into from
Dec 20, 2022

Conversation

danielduqma
Copy link
Contributor

@danielduqma danielduqma commented Oct 7, 2022

Migration of what was implemented for 14.0. This also fixes #868

Related: #849

@danielduqma danielduqma mentioned this pull request Oct 7, 2022
38 tasks
@sbidoul
Copy link
Member

sbidoul commented Oct 7, 2022

/ocabot migration pos_receipt_hide_price

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Oct 7, 2022
@danielduqma danielduqma force-pushed the 16.0-mig-receipt-hide-price branch 2 times, most recently from fb6caeb to 30bef8c Compare November 28, 2022 06:53
@ivantodorovich
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@ivantodorovich The rebase process failed, because command git push --force factorlibre tmp-pr-851:16.0-mig-receipt-hide-price failed with output:

remote: Permission to factorlibre/pos.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/factorlibre/pos/': The requested URL returned error: 403

@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Dec 19, 2022
Signed-off-by ivantodorovich
@OCA-git-bot
Copy link
Contributor

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

To https://github.com/OCA/pos
 ! [rejected]          16.0-ocabot-merge-pr-851-by-ivantodorovich-bump-nobump -> 16.0 (non-fast-forward)
error: failed to push some refs to 'https://***@github.com/OCA/pos'
hint: Updates were rejected because a pushed branch tip is behind its remote
hint: counterpart. Check out this branch and integrate the remote changes
hint: (e.g. 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@danielduqma danielduqma force-pushed the 16.0-mig-receipt-hide-price branch from 30bef8c to 3ee48db Compare December 20, 2022 07:20
@danielduqma danielduqma force-pushed the 16.0-mig-receipt-hide-price branch from 3ee48db to 647d545 Compare December 20, 2022 07:23
@danielduqma
Copy link
Contributor Author

Rebased over #882, merge should be OK now

@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 36ca0c0 into OCA:16.0 Dec 20, 2022
@OCA-git-bot
Copy link
Contributor

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

[14] pos_receipt_hide_price is useless, it hides the price but shows vat
6 participants