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

ProcessMetadata should have a single process identifier #640

Open
javierestevez opened this issue Nov 14, 2023 · 0 comments
Open

ProcessMetadata should have a single process identifier #640

javierestevez opened this issue Nov 14, 2023 · 0 comments
Labels
Under review Issues currently being reviewed

Comments

@javierestevez
Copy link

Hello,

The class ProcessMetadata can hold a list of process identifiers but per the Peppol spec there should only be one. I tried to see where the factory methods that take a list of process identifiers are used but they seem not used other than within the class itself and using Collections#singletonList.

Can we safely assume that there will always only be one value? If so, is this something you can foresee to fix in a upcoming major version?

Thank you,
Javier

@aaron-kumar aaron-kumar added the Under review Issues currently being reviewed label Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Under review Issues currently being reviewed
Projects
None yet
Development

No branches or pull requests

2 participants