Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Malformed Input #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Malformed Input #13

wants to merge 2 commits into from

Conversation

kevinvenclovas
Copy link

Hi,
now i got the issue after come back again.

You need to recode the WriteFragmentTo in the Claytondus.AmazonMWS.Products.Model.MoneyType class.
With the german localization the ".ToString" function convert the decimal "Amount" -> 12.99M to 12,99 but amazon need the dot version 12.99 .

So you need to change :
writer.Write("Amount", _amount);

to

writer.Write("Amount", _amount?.ToString(CultureInfo.InvariantCulture));

and it works for EUR. Please test it also for USD account. I can't test this without a USD account

@kevinvenclovas
Copy link
Author

@claytondus Can you merge this to master please?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant