Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Bugfix: TradeAllowanceCharges #7

Merged
merged 2 commits into from Jul 19, 2016
Merged

Bugfix: TradeAllowanceCharges #7

merged 2 commits into from Jul 19, 2016

Conversation

ChrisBlankDe
Copy link

Ich bin auf ein paar kleine Fehler gestoßen.
Zum einen war ein Indicator noch im alten Format und zum anderen fehlte die Currency bei einem Amount Feld welches mit zu wenig Nachkommastellen geliefert wurde.

@ChrisBlankDe
Copy link
Author

Bitte dann auch das nuget aktualisieren :)

@stephanstapel
Copy link
Owner

Vielen Dank für die Korrekturen. Hast Du das neue Erzeugungs-Ergebnis durch die ZUGFeRD-Validatoren geschickt, d.h. kommt ein korrekteres Ergebnis raus?

@ChrisBlankDe
Copy link
Author

Der DIN Validator nimmt die Änderung an als korrekt an bzw. die Fehlermeldungen kommen nicht mehr.

@stephanstapel stephanstapel merged commit 3c22bb3 into stephanstapel:master Jul 19, 2016
@stephanstapel
Copy link
Owner

nuget ist aktualisiert

@ChrisBlankDe
Copy link
Author

Danke. 👍

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

Successfully merging this pull request may close these issues.

None yet

2 participants