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.

Sign up for GitHub

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

Jump to bottom

Lieferschein Speichern u. schliessen "Als geschlossen markieren" #359

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Lieferschein Speichern u. schliessen "Als geschlossen markieren" #359

wants to merge 5 commits into from

Conversation

wernerhahn
Copy link
Contributor

behebt #702
behebt #694
TODO: callback überprüfen Lieferscheinliste hat kein callback

@z4m1n0
Copy link
Contributor

z4m1n0 commented Sep 17, 2024

Ich habe den Commit überarbeitet und die Logik zum Schließen in eine separate Methode ausgelagert. Dazu ist habe ich auch die Anzeige des Status verbessert, dieser wird jetzt vollständig angezeigt.

bblessmann
bblessmann reviewed Sep 18, 2024
View reviewed changes
Copy link
Member

@bblessmann bblessmann left a comment

Choose a reason for hiding this comment

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

Was mir nicht klar ist, warum geschlossene LS nicht gespeichert werden dürfen. War das vorher auch so? Was ist, wenn ich Bemerkungen oder interne Bemerkungen ändern will? Evtl. mal auf der Mailingliste diskutieren?

templates/design40_webpages/delivery_order/tabs/basic_data.html
</tr>
<td rowspan="2">
<span id="data-status-line" class="data wi-normal" style="overflow:visible;height:auto">
[% SELF.order.presenter.status_line || '&nbsp;' %]
Copy link
Member

Choose a reason for hiding this comment

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

Sollte hier nicht besser der style ins css (für diese Status-Zeile oder als Klasse)?

Copy link
Contributor

Choose a reason for hiding this comment

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

Erstellen wir für ein Element eine Klasse? Ich finde die Sichtbarkeit im html eigentlich gut. Gibt es im Projekt da Vorgaben oder Regeln?

Copy link
Member

Choose a reason for hiding this comment

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

Ich dachte mir, der style kommt evtl. öfter vor und könnte dann wiederverwendet werden. Ansonten evtl. mal mit rebootl besprechen.

@z4m1n0
Copy link
Contributor

z4m1n0 commented Sep 18, 2024

Was mir nicht klar ist, warum geschlossene LS nicht gespeichert werden dürfen. War das vorher auch so? Was ist, wenn ich Bemerkungen oder interne Bemerkungen ändern will? Evtl. mal auf der Mailingliste diskutieren?

Ich habe nachgeschaut und im alten Lieferschein konnten geschlossene auch gespeichert werden. Werde ich ändern.

@wernerhahn
Copy link
Contributor Author

Tamino hast du den etzten Punkt schon gemacht?

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

@bblessmann bblessmann Awaiting requested review from bblessmann

@z4m1n0 z4m1n0 Awaiting requested review from z4m1n0

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wernerhahn @z4m1n0 @bblessmann