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

billing_address support + color-only-structure #239

Merged
merged 6 commits into from
Jan 9, 2024

Conversation

TeXhackse
Copy link
Contributor

wie besprochen, @jbueren - guck mal bitte, ob ich alle Dateitypen erwischt habe, die Zuordnung bzgl. der Dateinamen ist nicht immer ganz so einfach.

Ich hab es außerdem gegenüber dem, was du bei $KUNDE gemacht hattest, die Struktur so umgestellt, dass auf die Variable nur einmal geprüft wird, müsste ja auch gehen, halte ich für übersichtlicher.

@TeXhackse TeXhackse requested a review from jbueren December 8, 2023 12:24
@@ -74,7 +73,7 @@
<%name%>\strut\\
<%if department_1%><%department_1%>\\<%end if%>%
<%if department_2%><%department_2%>\\<%end if%>%
<%cp_givenname%> <%cp_name%>\strut\\
<%cp_title%> <%cp_givenname%> <%cp_name%>\strut\\
Copy link
Contributor

Choose a reason for hiding this comment

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

Einrückung?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Die ist über die Dateien hinweg dann wohl nicht einheitlich, ich hab das nicht manuell gemacht, aber dann würde ich sie jetzt über alles hinweg vereinheitlichen… sonst ist das langfristig echt nervig.

@wernerhahn
Copy link
Contributor

Ist das gewollt, dass bei allen Belegtypen, die Rechnungsadresse genommen wird? Ich denke dies macht nur Sinn Bei den Zahlungsbelegen (Rechnung)invoice, (Gutschrift)credit_note,proforma,statement und check.

@TeXhackse
Copy link
Contributor Author

Ist das gewollt, dass bei allen Belegtypen, die Rechnungsadresse genommen wird? Ich denke dies macht nur Sinn Bei den Zahlungsbelegen (Rechnung)invoice, (Gutschrift)credit_note,proforma,statement und check

Ist sie nicht bei allen, ich glaube aber ich hab 1 zu viel. Aber genau deswegen die frage, und bitte gib mir wenndann Dateinamen, und nicht eure Benennung, weil genau das mein Problem ist, ich weiß nicht, wie der Knopf zu welcher Datei heißt.

@wernerhahn
Copy link
Contributor

Hier Rechnungsadresse:
templates/print/marei/invoice.tex
templates/print/marei/credit_note.tex
templates/print/marei/proforma.tex
templates/print/marei/zahlungserinnerung.tex
templates/print/marei/zahlungserinnerung_invoice.tex
templates/print/marei/statement.tex
templates/print/marei/check.tex

hier nicht
templates/print/marei/purchase_order.tex
templates/print/marei/sales_order.tex
templates/print/marei/sales_quotation.tex

@TeXhackse TeXhackse force-pushed the marei/LaTeX-adjustments branch from a97347b to dd43629 Compare December 8, 2023 13:20
@TeXhackse
Copy link
Contributor Author

Mosu meinte gerade dass die if Abfragen nicht schachtelbar sind. Ist das korrekt? Könnte das jemand checken?

@TeXhackse
Copy link
Contributor Author

Entwarnung, sollte passen.

@jbueren jbueren merged commit 3757e0e into master Jan 9, 2024
1 check passed
@jbueren jbueren deleted the marei/LaTeX-adjustments branch January 9, 2024 11:16
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.

3 participants