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

Integrate `invoice2data` #9

Closed
m3nu opened this Issue May 15, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@m3nu
Collaborator

m3nu commented May 15, 2018

After loading a PDF without XML embedded, the XML profile loaded from the template should be populated using values discovered by invoice2data.

This may need using a config-file or similar to set defaults. Like Seller's details.

@m3nu m3nu added the type:feature label May 15, 2018

@duskybomb

This comment has been minimized.

Show comment
Hide comment
@duskybomb

duskybomb Jun 30, 2018

Collaborator

Here what I have understood and think we should do:

  • discover values using invoice2data from PDF
  • read the config-file which contains two parts:
    • DEFAULTS: if there is some undiscovered value default values will be used
    • CUSTOM_SETTING: to override values discovered by invoice2data
  • Add values to the empty XML template which was loaded earlier.
Collaborator

duskybomb commented Jun 30, 2018

Here what I have understood and think we should do:

  • discover values using invoice2data from PDF
  • read the config-file which contains two parts:
    • DEFAULTS: if there is some undiscovered value default values will be used
    • CUSTOM_SETTING: to override values discovered by invoice2data
  • Add values to the empty XML template which was loaded earlier.

@m3nu m3nu closed this Jul 3, 2018

@duskybomb

This comment has been minimized.

Show comment
Hide comment
@duskybomb

duskybomb Jul 9, 2018

Collaborator

Why did you close this issue? Will this feature be exclusive to invoicex-gui?

Collaborator

duskybomb commented Jul 9, 2018

Why did you close this issue? Will this feature be exclusive to invoicex-gui?

@m3nu

This comment has been minimized.

Show comment
Hide comment
@m3nu

m3nu Jul 11, 2018

Collaborator

Yes. It's easy enough for users to just use both libs at the same time.

And it saves maintaining the integration. We are already very short on resources as it is.

Collaborator

m3nu commented Jul 11, 2018

Yes. It's easy enough for users to just use both libs at the same time.

And it saves maintaining the integration. We are already very short on resources as it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment