-
Notifications
You must be signed in to change notification settings - Fork 40
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
eForms support #42
Comments
Hi @jonatanprins no plans so far, but that seems to be a quick win... |
Hi @phax, that would definately benefit us. If it were to be added, how would this go? |
Well what I would do, from the top of my head, is something like this:
|
Alright, so we had a go at it. Ran into a some issues. EForms has custom versions for:
It would not generate the classes for them when it depends on ph-ubl23, so I omitted it for now and used the XSDs supplied by TED (https://github.com/OP-TED/eForms-SDK/tree/main/schemas). Browsing stackoverflow got me here: https://stackoverflow.com/questions/50176111/maven-jaxb2-plugin-reusing-commons-xsd-within-same-project. Not sure what the way to go is here. The other issue I ran into when running
Looks like the generated code uses a CommunicationType getter where, to make it compile, it should probably use the CommunicationTypeValue getter. I opened a PR. If you have time maybe you can have a look. |
@jonatanprins well, it was a bit of pain in the a but I think I finally managed to get it working. Please see the https://github.com/phax/ph-ubl/tree/eforms branch for details. What I did: depend on the ph-ubl23 project - that gives the most synergies and avoids "double code" generation. What I did instead is to create a helper class Merry christmas |
Hello @phax, First of all: thank you for your answer/fix. Hope you've had a nice christmas! In @jonatanprins his absence.. i'm giving it a whirl (co-worker). Looks like all is compiling and running fine. The helper class will suffice for now 👍. Having some troubles with running What to do next? how can we promote to master? which hurdles to take :) |
@DKamerbeek thanks for your feedback. The problem with IntelliJ is, that the base directory differs from the Eclipse one. If the base directory is set to the project directory and not If this suffices, I will just merge it :) |
Yes this is fine, thanks in advance |
Part of the 6.7.0 release. Thanks for your contribution! |
We are working on a preliminary implementation for handling invoices based on (TED) https://github.com/OP-TED/eForms-SDK. They've built their documents on the UBL23 standard. We'd like to use ph-ubl for reading and creating of the documents.
Are there any plans to support eForms? Maybe an extra package like you have for DIAN or PE?
Regards,
The text was updated successfully, but these errors were encountered: