[IMP] payment_provider/mercado pago: content updates #5525
Conversation
samueljlieber
changed the title
samueljlieber
force-pushed
the
masi-odoo-patch-2
branch
from
57040c5 to
8338ae2
Compare
|
Hi @masi-odoo I've corrected the commit msg and PR title, as well as a made a few syntax corrections in the RST. I noticed the images are missing in this PR, can you please link them here so I can add them 🙂 |
|
Hello @samueljlieber thanks for that! Linking Google Drive folder with images: https://drive.google.com/drive/folders/1CuOVMkXwREj_H5yuKdJ1oPksQWoDJ6Nk?usp=sharing |
samueljlieber
force-pushed
the
masi-odoo-patch-2
branch
from
8338ae2 to
1d92f29
Compare
|
Hi @masi-odoo I've implemented your images and updated the formatting for your changes in 1d92f29. |
AntoineVDV
left a comment
AntoineVDV
left a comment
There was a problem hiding this comment.
Hello, I left some review comments below :)
content/applications/finance/payment_providers/mercado_pago.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/payment_providers/mercado_pago.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/payment_providers/mercado_pago.rst
Outdated
Show resolved
Hide resolved
samueljlieber
force-pushed
the
masi-odoo-patch-2
branch
2 times, most recently
from
c00e36c to
b8dff55
Compare
|
Thank you for your review @AntoineVDV! I implemented your changes in c00e36c and b8dff55. I also added a seealso link to the latest Mercado Pago Odoo webinar for @masi-odoo. |
auva-odoo
left a comment
auva-odoo
left a comment
There was a problem hiding this comment.
Hey @masi-odoo thank you for you work! I just added 2 suggestions, let me know if you have any questions!
content/applications/finance/payment_providers/mercado_pago.rst
Outdated
Show resolved
Hide resolved
content/applications/finance/payment_providers/mercado_pago.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: samueljlieber <sali@odoo.com> This commit will add important clarifications to use and test the Mercado Pago payment provider and add visual documentation similar to other provider's documentation
samueljlieber
force-pushed
the
masi-odoo-patch-2
branch
from
b8dff55 to
2d1ae8a
Compare
|
Thank you @auva-odoo for your review, I implemented the changes in 2d1ae8a on behalf of @masi-odoo! |
auva-odoo
left a comment
auva-odoo
left a comment
There was a problem hiding this comment.
Thanks @samueljlieber ! :)
@robodoo r+
This commit will add important clarifications to use and test the Mercado Pago payment provider and add visual documentation similar to other provider's documentation closes #5525 Signed-off-by: Audrey Vandromme (auva) <auva@odoo.com> Co-authored-by: samueljlieber <sali@odoo.com>
5 participants
This commit will add important clarifications to use and test the Mercado Pago payment provider and add visual documentation similar to other provider's documentation