Skip to content

[ADD][14.0] account_payment_order_vendor_email #776

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
merged 1 commit into from
Feb 24, 2022
Merged

[ADD][14.0] account_payment_order_vendor_email #776

merged 1 commit into from
Feb 24, 2022

Conversation

@Murtaza-OSI
Copy link

No description provided.

@max3903 max3903 added this to the 14.0 milestone Mar 7, 2021
@max3903 max3903 force-pushed the 14.0-MIG-account_payment_order_vendor_email branch 2 times, most recently from dc0f34c to 4a1243e Compare March 10, 2021 19:49
@max3903 max3903 changed the title [ADD][14.0] account_payment_order_vendor_email Module. [ADD][14.0] account_payment_order_vendor_email Mar 13, 2021
@max3903 max3903 self-assigned this Mar 13, 2021
@atchuthan atchuthan mentioned this pull request Jun 10, 2021
Merged
@atchuthan
Copy link
Member

Please add the changes in ursais#7 as the tablet format is improved and fix for current issue of when editing the template, the sent email is empty.

@Chandresh-SerpentCS Chandresh-SerpentCS force-pushed the 14.0-MIG-account_payment_order_vendor_email branch 2 times, most recently from 83a2825 to c3de07b Compare September 16, 2021 19:02
@Chandresh-OSI Chandresh-OSI force-pushed the 14.0-MIG-account_payment_order_vendor_email branch 2 times, most recently from 8b5876b to 37144c2 Compare September 24, 2021 13:20
patrickrwilson
patrickrwilson approved these changes Sep 24, 2021
View reviewed changes
Copy link

@patrickrwilson patrickrwilson left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@atchuthan atchuthan mentioned this pull request Sep 28, 2021
Closed
@dreispt
Copy link
Member

dreispt commented Sep 28, 2021

Hello, sorry for coming late here: why is this specific to the US.
It look like this can be useful for other countries too.
Can we consider proposing it to OCA/account payment instead?

@dreispt
Copy link
Member

dreispt commented Oct 5, 2021

@max3903 are you OK to move this?

@dreispt
Copy link
Member

dreispt commented Nov 9, 2021

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-776-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 9, 2021
@OCA-git-bot
Merge PR #776 into 14.0
58ada1a 
Signed-off-by dreispt
@OCA-git-bot
Copy link
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-776-by-dreispt-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.

@dreispt dreispt force-pushed the 14.0-MIG-account_payment_order_vendor_email branch from 37144c2 to c2ba5bf Compare November 9, 2021 14:06
@dreispt dreispt force-pushed the 14.0-MIG-account_payment_order_vendor_email branch 2 times, most recently from 5111935 to 1a0c406 Compare November 9, 2021 14:24
@dreispt dreispt removed the question label Nov 9, 2021
atchuthan
atchuthan suggested changes Nov 9, 2021
View reviewed changes
@Murtaza-OSI Murtaza-OSI force-pushed the 14.0-MIG-account_payment_order_vendor_email branch from 1a0c406 to e0218bc Compare November 23, 2021 07:30
@Murtaza-OSI Murtaza-OSI force-pushed the 14.0-MIG-account_payment_order_vendor_email branch from e0218bc to b672ada Compare November 23, 2021 07:33
@dreispt
Copy link
Member

dreispt commented Feb 24, 2022

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 3e51a3e into OCA:14.0 Feb 24, 2022
@OCA-git-bot
Copy link
Contributor

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

agyamuta pushed a commit to ursais/bank-payment that referenced this pull request Jul 27, 2023
@OCA-git-bot @agyamuta
Merge PR OCA#776 into 14.0
b3aa83b 
Signed-off-by dreispt

(cherry picked from commit 3e51a3e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@atchuthan atchuthan atchuthan requested changes

@patrickrwilson patrickrwilson patrickrwilson approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@max3903 max3903

Labels

enhancement merged 🎉

Projects

None yet

Milestone

14.0

Development

Successfully merging this pull request may close these issues.

6 participants

@Murtaza-OSI @atchuthan @OCA-git-bot @dreispt @patrickrwilson @max3903