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

2024 q2 member review #150

Merged
merged 26 commits into from
May 22, 2024
Merged

2024 q2 member review #150

merged 26 commits into from
May 22, 2024

Conversation

jerouris
Copy link
Contributor

No description provided.

Jelte Jansen and others added 26 commits September 20, 2023 14:52
NL-R-007 requires that a payment means is provided, which only makes sense if there is a payment, i.e. for positive invoices.
NL-R-007 requires that a payment means is provided, which only makes sense if there is a payment, i.e. for positive invoices (typecode != 381).
Addendum to the fix in a7a6898, in case the CreditNote itself is negative, the payment means should be required.
Addendum to the fix in 5cc83d5, in case the CreditNote itself is negative, the payment means should be required.
Add additional error cases for the fixes in commits 7b5032f and edc5743
POACC-735, missed one thing in the release notes
#POACC-730
@jerouris jerouris merged commit 30bf64e into master May 22, 2024
1 check passed
@jerouris jerouris deleted the 2024-q2-member-review branch September 18, 2024 08:53
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.

3 participants