Problem/Motivation

Here we have hardcode the locale as `ES-ar`

https://github.com/mercadopago/sdk-js#mercadopagopublic_key-options

Proposed resolution

Please get the value from a config and research on the MercadoPago API expected values.

Issue fork commerce_mercadopago-3368693

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Zarpele created an issue. See original summary.

fabianfiorotto’s picture

fabianfiorotto commented

I tried changing that value in the code and it seems like Mercado Pago ignores that parameter. I'd consider deleting it from the code.

zarpele’s picture

zarpele commented
Issue summary: View changes

Zarpele opened merge request !2

zarpele’s picture

zarpele commented
Status: Active » Needs review

@fabianfiorotto I agree with that and it passes my local testing.

I filed up a new MR, lets the API define the locale based on the browser. More Info: https://github.com/mercadopago/sdk-js#params

  • 0956fa2a committed on master 
    Issue #3368693: Locale from config, removing the locale param.
fabianfiorotto’s picture

fabianfiorotto commented
Status: Needs review » Fixed

  • 0956fa2a committed on 1.0.x 
    Issue #3368693: Locale from config, removing the locale param.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.