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

[14.0] [mig]pos_fixed_discount #692

Closed
wants to merge 13 commits into from

Conversation

hkapatel-initos
Copy link
Contributor

No description provided.

@hkapatel-initos hkapatel-initos mentioned this pull request Jul 29, 2021
16 tasks
@hkapatel-initos hkapatel-initos force-pushed the 14.0-mig-pos_fixed_discount branch 2 times, most recently from 45d980a to ecb2eb7 Compare August 5, 2021 08:47
pos_fixed_discount/static/src/js/discount.js Outdated Show resolved Hide resolved
pos_fixed_discount/static/src/js/discount.js Outdated Show resolved Hide resolved
pos_fixed_discount/static/src/js/discount.js Outdated Show resolved Hide resolved
@hkapatel-initos hkapatel-initos force-pushed the 14.0-mig-pos_fixed_discount branch 2 times, most recently from 61d28e3 to 45e32bb Compare August 6, 2021 10:15
Copy link
Contributor

@fshah-initos fshah-initos left a comment

Choose a reason for hiding this comment

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

LGTM

@ivantodorovich
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Aug 6, 2021
Signed-off-by ivantodorovich
@ivantodorovich
Copy link
Contributor

I don't know what you pushed, but the bot was already merging 😅 I'm not sure what's gonna happen now, maybe it'll see the change and cancel

If not, create a new PR after this one's merged pls

@OCA-git-bot
Copy link
Contributor

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

7 participants