Skip to content

Conversation

@andg-odoo
Copy link
Contributor

Mercado Pago requires the X-platform-id header to be passed in all requests to their platform to keep track of the amount of odoo customers that exist.

This key is not secret and perfectly fine to be committed in this repo without worry.

task-4628319

@robodoo
Copy link
Contributor

robodoo commented Mar 7, 2025

Pull request status dashboard

@C3POdoo C3POdoo added the RD research & development, internal work label Mar 8, 2025
@masi-odoo
Copy link
Contributor

Hello @andg-odoo I did testing and the PR works great.

Mercado Pago also confirmed that they are receiving the platform_id correctly.

LGTM! This can be reviewed

Copy link
Contributor

@livaios livaios 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,

Thanks for your work on this.
Just one small nitpicking comments 🙂

Have a nice day!

@livaios livaios marked this pull request as ready for review March 19, 2025 13:16
@C3POdoo C3POdoo requested a review from a team March 19, 2025 13:29
@livaios livaios requested review from a team and removed request for a team March 19, 2025 13:55
@andg-odoo andg-odoo force-pushed the 16.0-payment_mercado_pago-add-platform-id-4628319-andg branch from 08e4fae to d7b44f6 Compare March 19, 2025 15:50
@AntoineVDV AntoineVDV removed the request for review from a team March 19, 2025 16:51
Copy link
Contributor

@AntoineVDV AntoineVDV left a comment

Choose a reason for hiding this comment

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

@robodoo
Copy link
Contributor

robodoo commented Mar 19, 2025

@AntoineVDV you may want to rebuild or fix this PR as it has failed CI.

Mercado Pago requires the X-platform-id header to be passed in all
requests to their platform to keep track of the amount of odoo
customers that exist.

This key is not secret and perfectly fine to be committed in this
repo without worry.

task-4628319
@andg-odoo andg-odoo force-pushed the 16.0-payment_mercado_pago-add-platform-id-4628319-andg branch from d7b44f6 to 31da97a Compare March 19, 2025 17:08
@andg-odoo
Copy link
Contributor Author

Test failed due to branch being behind head, rebasing it should fix it due to this Enterprise PR from two days ago.

@AntoineVDV
Copy link
Contributor

@robodoo r+

robodoo pushed a commit that referenced this pull request Mar 19, 2025
Mercado Pago requires the X-platform-id header to be passed in all
requests to their platform to keep track of the amount of odoo
customers that exist.

This key is not secret and perfectly fine to be committed in this
repo without worry.

task-4628319

closes #200864

Signed-off-by: Antoine Vandevenne (anv) <anv@odoo.com>
@robodoo robodoo closed this Mar 19, 2025
@fw-bot fw-bot deleted the 16.0-payment_mercado_pago-add-platform-id-4628319-andg branch March 26, 2025 19:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants