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

EN 16931 tags #394

Closed
wants to merge 5 commits into from
Closed

EN 16931 tags #394

wants to merge 5 commits into from

Conversation

apardods
Copy link
Contributor

Added tags for GOBL objects mapping them to their correspondent EN 16931 Business Term (BT) or Business Group (BG).

Copy link
Contributor

@cavalle cavalle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!! I had exactly the same idea a few days ago

Copy link
Collaborator

@samlown samlown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code itself looks sound, but I'm afraid this isn't the best approach. The fundamental issue is that the EN16931 semantic codes usually have a context, whereas the syntax defined in the GOBL structures are designed to be used anywhere.

The objective however is sound, its just the approach that needs changing. In my opinion there should be a table of EN16931 codes to JSON pointers/paths of attributes inside GOBL structures so that you always have a context. This table should also define "notes" or something similar to describe any additional implications or meanings. The mapping table should also be adjusted when addons or regimes are applied.

This might be a good subject to brainstorm next week.

@cavalle
Copy link
Contributor

cavalle commented Oct 16, 2024

It's a good point that GOBL structs have a scope that could potentially exceed that of EN16391 and putting the metadata directly in the structs, while convenient, may not makes any sense in other contexts.

It's also true that mapping may not always be direct or trivial and additional notes to indicate special details may also be useful or necessary.

@urakozz
Copy link

urakozz commented Oct 20, 2024

Thank you for this PR, it's more comprehensive than FacturX documentation 😄

@samlown
Copy link
Collaborator

samlown commented Oct 25, 2024

Closing this one in favor of #401

@samlown samlown closed this Oct 25, 2024
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.

4 participants