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

Bug/845 ch label #884

Open

clean-coder wants to merge 2 commits into main from bug/845_ch_label

Collaborator

clean-coder commented Apr 8, 2024

Es ist nicht nur CHF falsch, sondern auch EUR und FTE. Dies ist mal ein fix, dass CHF/EUR/FTE ok ausschauen.

Fragen:

UnitValueTransformationPipe.ts

warum ist CHF und Euro verschieden, d.h. warum wird addCHFSign gebraucht? Warum ".-" bei CHF und nicht bei EUR?
irgendwie unglücklich, dass man eine Klasse für Labels hat, aber hier Teile der Labels wieder handcodiert

UnitLabelTransformationPipe.ts:

ist eigentlich das Label zur Unit, aber es gibt das Problem, dass das Label je nach Unit vor oder nach der Unit stehen sollte (nur die Klasse weiss das nicht)
aktuelle Implementation macht wenig sinn, denn es wird immer "" zurückgegeben um das "Label vor/nach Unit" zu handeln

Vorschlag:

mal ein Story machen, wo das das Zusammenspiel der Value, Label und Html verbessert wird

clean-coder and others added 2 commits

April 8, 2024 08:18


          fix Labels CHF, EUR, FTE

e87de5e


          add 'GJ 23/24-Q4' in H2 setup data

8fc0a95

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