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

Add adjusted template for invoices #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukaspanni
Copy link

Added an additional template for invoices based on the letter template.
Could also be combined into one, but I'm unsure on what the better solution would be.

@benedictdudel
Copy link
Owner

Hi 👋

thank you for your contribution ❤️ I really love the idea of a template for invoices. I will come back to you in the next few days!

@benedictdudel
Copy link
Owner

Over the last couple of days, I had done a refactoring of the template. The problem with the template before was that a lot of settings were missing which the official default template of pandoc included. The template is now based on the official and has just little tweaks for the scrlttr2 document class.

The project now has a dedicated examples directory where multiple letter variants can be published. I already started with an invoice and if you like you could add your footer there. You should use a custom .lco file for the footer. You can find more information here.

@lukaspanni
Copy link
Author

The new options look great!
I am going to look into providing an update so that it works for my workflow.
Will probably postpone to next year.

@lukaspanni
Copy link
Author

I updated the invoice-footer and introduced a variable to toggle it.
I believe that this makes using it easier for users since they don't have to worry about having the .lco file in the right directory.
If you don't agree I will make a .lco available

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.

2 participants