Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR allows us to set a business process id and specification as part of the API, allowing us to fill the tags:
ram:BusinessProcessSpecifiedDocumentContextParameter(previously not set)ram:GuidelineSpecifiedDocumentContextParameter(previously set depending on the version parameter only, not changeable from the outside)It does not set any of these tags by default, so there are no changes for exiting users.
The PR adds two constants for possible values. These maps could be extended later, if compatibility with other standards is possible.
This is partial support for the XRechnung CIUS. I will open an issue shortly and link this PR from there.
Please don't hesitate to make any adjustments you need or reject if it does not fit.