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

202405 feat create assembly extended #334

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

Conversation

wernerhahn
Copy link
Contributor

Fertigen mit Chargenauswahl
Mehr informationen im Titen Wieviel und Was gefertigt werden soll.
Mindesthaltbarkeitsdatum in Tabelle mit aufgenommen

@wernerhahn wernerhahn requested a review from bblessmann May 22, 2024 14:51
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.

Die Anzeige vom MHD in der Auswahlmaske funktioniert nicht. Hast Du das probiert? Da muss dann mindestens in bin/mozilla/wh.pl bei get_stock (by => ...) (ca. Zeile 405/406) bestbefore mit aufgenommen werden (oder evtl. 'for_allocate'). Ich hatte da einen kleinen Kommentar hinterlassen.
Das muss dann aber für alle Fälle getestet werden, z.B. MHD an/aus; verschiedene Chargen mit MHD und ohne und eine Charge (oder ohne Chargennummer) mit Lagerbeständen mit, mit verschiedenen und ohne MHD, usw.

@@ -35,7 +38,7 @@ <h1>[% HTML.escape(title) %]</h1>
[% FOREACH part_id = stocked_by_parts_id.keys.sort -%]
[% NEXT UNLESS stocked_by_parts_id.$part_id.size > 1 %]
<tr class="listheading">
<td colspan="6">[% 'needed qty' | $T8 %]</td>
<td colspan="7">[% 'needed qty' | $T8 %]</td>
Copy link
Member

Choose a reason for hiding this comment

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

Das colspan muss auch in Abhängigkeit zu INSTANCE_CONF.get_show_bestbefore geändert werden, sonst stimmt es nicht, wenn MHD aus ist.
Gleiches im alten Design.

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