search

PrestaShopCorp / mercadopagobr

2 stars 9 forks source link

[Urgent Fix] - Removing public key validation, module will stop working if not removed #13

Closed ricardobrito closed 9 years ago

Quetzacoalt91 commented 9 years ago

Hi @ricardobrito,

I'm sorry but I don't have any four digits sent in the request: capture du 2015-05-12 18 00 14

Can you investigate ?

Regards

ricardobrito commented 9 years ago

Hi @Quetzacoalt91 ,

It seems there is an error with your payment since you should also send card_token_id. The card_token_id is only generated when all checkout data is ok.

Your post should be like: image

Another hypothesis is javascript cache - can you also check if your javascript has the four digits input? You should find this snippet on web inspector: image

Regards

ricardobrito commented 9 years ago

Let me know if you need credentials and testing credit cards. Thanks

Ricardo Brito Software Engineer

2015-05-12 13:52 GMT-03:00 Ricardo Brito de Souza < ricardo.brito@mercadopago.com.br>:

Hi Thomas,

It seems there is an error with your payment since you should also send card_token_id. The card_token_id is only generated when all checkout data is ok.

Your post should be like:

[image: Imágenes integradas 1]

Another hypothesis is javascript cache - can you also check if your javascript has the four digits input? You should find this snippet on web inspector: [image: Imágenes integradas 2]

Regards,

Ricardo Brito Software Engineer

2015-05-12 13:01 GMT-03:00 Thomas N notifications@github.com:

Hi @ricardobrito https://github.com/ricardobrito,

I'm sorry but I don't have any four digits sent in the request: [image: capture du 2015-05-12 18 00 14] https://cloud.githubusercontent.com/assets/6768917/7591928/edba04e4-f8d0-11e4-8bbb-903f5fea5f6f.png

Can you investigate ?

Regards

— Reply to this email directly or view it on GitHub https://github.com/PrestaShop/mercadopagobr/pull/13#issuecomment-101331040 .

Quetzacoalt91 commented 9 years ago

Hi @ricardobrito,

I have the latest JS on my shop, nothing wrong on this side.

In case of error, I guess you would like to not send the form ? Because this is what I see on my side (my payment information are wrong, FYI)

Regards

ricardobrito commented 9 years ago

Hi @Quetzacoalt91

Can you provide me the form data you are using so I can reproduce it?

Also, this behaviour does already exist - this is not caused by this commit.

Can I create an enhancement to work on it afterwards?

Quetzacoalt91 commented 9 years ago

Okay, the latest commit allowed me to complete the payment process.

I will now merge the PR, thanks.