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

Master payment payulatam payment integration nla #24802

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
7 participants
@nla-odoo
Copy link
Contributor

nla-odoo commented May 18, 2018

Added a new payment acquirer PayUlatam.

Task: https://www.odoo.com/web#id=1826303&action=327&model=project.task&view_type=form&menu_id=4720

Pad: https://pad.odoo.com/p/r.589bf0ced97b289b18d5965d8cfaec64

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch 2 times, most recently May 18, 2018

@C3POdoo C3POdoo added the RD label May 22, 2018

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch 4 times, most recently May 24, 2018

'views/payment_payulatam_templates.xml',
'data/payment_acquirer_data.xml',
],
}

This comment has been minimized.

@mreficent

mreficent May 30, 2018

Contributor

I think you should add 'post_init_hook': 'create_missing_journal_for_acquirers', like in the others payment_x modules

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch 21 times, most recently May 31, 2018

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch Jun 11, 2018

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch 3 times, most recently Jun 19, 2018

@mba-odoo

This comment has been minimized.

Copy link
Contributor

mba-odoo commented Jun 19, 2018

LGTM

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch to 0d32a37 Jun 19, 2018

@jacobus101

This comment has been minimized.

Copy link

jacobus101 commented Sep 13, 2018

Is this gonna be in v12?

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch from 0d32a37 Nov 26, 2018

@robodoo robodoo added the seen 🙂 label Nov 26, 2018

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch to 1aab709 Nov 26, 2018

@robodoo robodoo added the CI 🤖 label Nov 26, 2018

@tivisse tivisse force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch from 1aab709 to 73f901c Feb 11, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 11, 2019

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch from 73f901c to 7972c4d Feb 11, 2019

@robodoo robodoo removed the CI 🤖 label Feb 11, 2019

@nla-odoo nla-odoo force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch from 7972c4d to 67add26 Feb 11, 2019

@robodoo robodoo added the CI 🤖 label Feb 11, 2019

@tivisse tivisse force-pushed the odoo-dev:master-payment_payulatam-payment-integration-nla branch from 67add26 to b0eb01b Feb 11, 2019

@robodoo robodoo removed the CI 🤖 label Feb 11, 2019

@tivisse

This comment has been minimized.

Copy link
Contributor

tivisse commented Feb 11, 2019

robodoo pushed a commit that referenced this pull request Feb 11, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Feb 11, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 11, 2019

@tivisse tivisse deleted the odoo-dev:master-payment_payulatam-payment-integration-nla branch Feb 12, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment