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

[ADD] payment_providers: Nuvei #11157

Closed
wants to merge 1 commit into from
Closed

Conversation

andg-odoo
Copy link
Contributor

Documentation for Nuvei payment provider.

task-4172642

See also:

@robodoo
Copy link
Collaborator

robodoo commented Oct 8, 2024

Pull request status dashboard

@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 4 times, most recently from 4c6c067 to 5892751 Compare October 15, 2024 16:01
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 2 times, most recently from 5f940af to 7fc8728 Compare October 21, 2024 20:38
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 2 times, most recently from b52ca50 to fde1915 Compare November 27, 2024 22:30
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from fde1915 to 9d45fab Compare December 5, 2024 18:00
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 9d45fab to 483b469 Compare December 17, 2024 17:29
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 2 times, most recently from 98ccb9d to ebd8218 Compare January 13, 2025 21:48
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from ebd8218 to 8d7b23e Compare January 21, 2025 21:08
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 2 times, most recently from aed8451 to 3b4361d Compare February 5, 2025 16:56
@andg-odoo andg-odoo marked this pull request as ready for review February 5, 2025 16:56
@C3POdoo C3POdoo requested a review from a team February 5, 2025 16:58
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 3b4361d to 05d148b Compare February 7, 2025 21:29
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Hello @andg-odoo and thank you for your work! I added a few comments and suggestions; feel free to reach out if you have any questions. (Apologies in advance if any of them are inaccurate; I wasn’t able to test the configuration myself, so it was a bit challenging.)

@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 05d148b to dbd8930 Compare February 11, 2025 22:50
@andg-odoo
Copy link
Contributor Author

@auva-odoo Thanks for the review, I agree with you on all cases, and have updated the documentation as requested.

@auva-odoo auva-odoo force-pushed the master-payment_nuvei-andg branch from dbd8930 to 946ae73 Compare February 14, 2025 10:53
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Thank you for the changes @andg-odoo, I force-pushed a few additional (minor) ones, I hope you don't mind. Thanks again!

@andg-odoo
Copy link
Contributor Author

andg-odoo commented Feb 14, 2025

Thanks for the once over @auva-odoo, we were considering adding a table to the page showing what payment methods Nuvei supports in what specific countries, would something like this (Either top or bottom) fit the style:
image

I'll try and get that written up today and push it. (We are more leaning towards the bottom style with countries on the top)

EDIT: Honestly, we tested it, and it's very bare with every country only supporting a select few providers (As seen below)
image

Probably won't do this this round but I feel it might be a useful thing for sales people to know which providers are supported so still open to ideas for future improvements.

The US is off Monday but I'll be back Tuesday for any additional comments/changes that are necessary.

@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch 4 times, most recently from 8d8915d to 7ad7a35 Compare February 24, 2025 18:32
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for the update @andg-odoo! I added some suggestions for improvement.

Comment on lines 50 to 51
Providers By Country
====================
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Providers By Country
====================
Payment methods
===============

I think you meant payment methods instead of providers?
(And we use sentence case in headings)

@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 7ad7a35 to 4281865 Compare February 25, 2025 17:33
@andg-odoo
Copy link
Contributor Author

@auva-odoo Thanks for the review, I have made the two adjustments.

@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 4281865 to 44057da Compare March 6, 2025 16:37
Copy link
Contributor

@auva-odoo auva-odoo left a comment

Choose a reason for hiding this comment

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

Thanks for the changes @andg-odoo !

@robodoo delegate+

Documentation for Nuvei payment provider.

task-4172642
@andg-odoo andg-odoo force-pushed the master-payment_nuvei-andg branch from 44057da to 60e2dc6 Compare March 7, 2025 19:09
@AntoineVDV
Copy link
Collaborator

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Mar 17, 2025

@andg-odoo @AntoineVDV linked pull request(s) odoo/odoo#181459 not ready. Linked PRs are not staged until all of them are ready.

robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 17, 2025
Nuvei is an online payment platform for the LATAM area. This integration
currently supports Argentina, Brazil, Canada, Chile, Colombia, Ecuador,
Mexico, Peru, United States, and Uruguay.

Nuvei supports card transactions as well as other local payment methods
unique per country.

task-4172642

closes #181459

Related: odoo/documentation#11157
Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
robodoo pushed a commit that referenced this pull request Mar 17, 2025
Documentation for Nuvei payment provider.

task-4172642

closes #11157

Related: odoo/odoo#181459
Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Mar 17, 2025
Nuvei is an online payment platform for the LATAM area. This integration
currently supports Argentina, Brazil, Canada, Chile, Colombia, Ecuador,
Mexico, Peru, United States, and Uruguay.

Nuvei supports card transactions as well as other local payment methods
unique per country.

task-4172642

closes #181459

Related: odoo/documentation#11157
Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
@robodoo robodoo closed this in 2f90bc7 Mar 17, 2025
gamarino pushed a commit to numaes/numa-public-odoo that referenced this pull request Mar 17, 2025
Nuvei is an online payment platform for the LATAM area. This integration
currently supports Argentina, Brazil, Canada, Chile, Colombia, Ecuador,
Mexico, Peru, United States, and Uruguay.

Nuvei supports card transactions as well as other local payment methods
unique per country.

task-4172642

closes odoo/odoo#181459

Related: odoo/documentation#11157
Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
omar-sherif9992 pushed a commit to odoo-dev/odoo that referenced this pull request Mar 26, 2025
Nuvei is an online payment platform for the LATAM area. This integration
currently supports Argentina, Brazil, Canada, Chile, Colombia, Ecuador,
Mexico, Peru, United States, and Uruguay.

Nuvei supports card transactions as well as other local payment methods
unique per country.

task-4172642

closes odoo#181459

Related: odoo/documentation#11157
Signed-off-by: Antoine Vandevenne (anv) <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants