Mehrere Änderungen (Robustere HTML anzeige / config.ini -> gitignore / Anzeige für Draft documente / bugfix customer address null ) #25
Conversation
|
@Perlenjaeger danke für die Änderungsvorschläge. Es würde mir die Arbeit deutlich erleichtern, wenn du für jedes der einzelnen Thema einen separaten PR aufmachst und die Commit-Messages und PR-Titel nach dem Conventional Commit-Schema benennst. Dann hätte ich einige PRs bereits mergen können, während wir für die anderen in Ruhe offene Fragen klären und sie ggf. wieder schließen könnten. So muss ich mir alle Änderungen auf einmal anschauen und entweder gesammelt ablehnen oder akzeptieren. |
| {% set from_date = doc.from_date %} | ||
| {% set to_date = doc.to_date %} | ||
| {% endif %} | ||
| {% set timesheet = frappe.get_doc("Timesheet", doc.timesheets.time_sheet)%} | ||
| {% if timesheet.start_date and timesheet.end_date %} | ||
| {% set from_date = timesheet.start_date %} | ||
| {% set to_date = timesheet.end_date %} | ||
| {% endif %} |
There was a problem hiding this comment.
Ich denke das sollte nicht Aufgabe des Druckformats sein. Abgesehen davon verwenden wir beispielsweise eine andere Logik (es wird immer ein ganzer Monat vom ersten bis letzten Tag abgerechnet). Besser ein Server Script erstellen, das die entsprechenden Werte nach Belieben in der Rechnung setzt.
|
Ja ich war zu faul. Werde das morgen machen. Ja das mit der Abrechnung stimmt.On 15 Oct 2024 22:26, Raffael Meyer ***@***.***> wrote:
@barredterra commented on this pull request.
In print_format/sales_invoice.jinja:
+ {% set from_date = doc.from_date %}
+ {% set to_date = doc.to_date %}
+ {% endif %}
+ {% set timesheet = frappe.get_doc("Timesheet", doc.timesheets.time_sheet)%}
+ {% if timesheet.start_date and timesheet.end_date %}
+ {% set from_date = timesheet.start_date %}
+ {% set to_date = timesheet.end_date %}
+ {% endif %}
Ich denke das sollte nicht Aufgabe des Druckformats sein. Abgesehen davon verwenden wir beispielsweise eine andere Logik (es wird immer ein ganzer Monat vom ersten bis letzten Tag abgerechnet). Besser ein Server Script erstellen, das die entsprechenden Werte nach Belieben in der Rechnung setzt.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
[
{
***@***.***": "http://schema.org",
***@***.***": "EmailMessage",
"potentialAction": {
***@***.***": "ViewAction",
"target": "#25 (review)",
"url": "#25 (review)",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
***@***.***": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]
|
Bei mir in ERPnext wurden die documente nicht richtig in der html vorschau angezeigt und auch die pdfs passten nicht mit den header und footer.
da im html und im pdf nicht immer die gleichen css parameter funktionieren musste ich eine ausnahme beim footer hinzufügen. der rest wurde deutlich stabieler als ich zusätzlich die breite definiert hatte und nicht nur auf die ränder verwies.
config.ini -> gitignore
es ist besser eine datei mit passwort inhalt in die ignore zu legen und ein beispiel zu bieten. die gefahr einmal sein passwort hoch zu puschen ist zu groß.
ich wollte eine anzeige eines Draft/Entworfs in der anschreiben zeile damit nicht aus versehen einmal ein falsches raus gesendet wird.
wenn keine address beim customer angegeben ist, funktionierte die vorlage nicht.
zusätzlich wird jetzt auch die performance perios (Leistungszeitraum) angezeigt, wenn eine stundenabrechnung dabei ist
Danke für deine tolle Arbeit.