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

Introduce Modelattribue isBacklog #1131

Open
4 tasks
kcinay055679 opened this issue Nov 7, 2024 · 1 comment · May be fixed by #1334
Open
4 tasks

Introduce Modelattribue isBacklog #1131

kcinay055679 opened this issue Nov 7, 2024 · 1 comment · May be fixed by #1334
Assignees

Comments

@kcinay055679
Copy link
Collaborator

kcinay055679 commented Nov 7, 2024

Momentan wird an mehreren Stellen im Frontend ein Regex genutzt um zu entscheiden ob ein Quartal ein Backlog-Quartal ist oder nicht. Neu soll das Backend stattdessen bei jedem Quartal ein Property mitgeben, welches die selbe Funktion erfüllt, i.e. isBacklog: true.

Die Regexe sind mindestens im objective-form-components.ts und error-interceptor.service.ts zu finden, könnten sich aber auch noch an deren Orten verstecken.

Anforderungen

  • Das Frontend soll nicht mehr über einen Regex entscheiden ob ein Quartal ein Backlog-Quartal ist
  • Das Backend soll die Information über die DTOs mitgeben

Aktzeptanzkriterien

  • Im Frontend wird kein Regex mehr für das Entscheiden ob ein Quartual ein Backlog-Quartal ist gebraucht
  • Das Backend hat ein entsprechendes Property bei allen DTOs wo es benötigt wird
  • Unit/Int-Test im Backend wurden ergänzt
  • Unit-Tests im Frontend wurden angepasst
@kcinay055679 kcinay055679 added the request New feature request label Nov 7, 2024
@peggimann peggimann removed their assignment Nov 20, 2024
@MasterEvarior MasterEvarior removed the request New feature request label Nov 20, 2024
@Miguel7373 Miguel7373 self-assigned this Jan 23, 2025
@Miguel7373 Miguel7373 linked a pull request Jan 23, 2025 that will close this issue
@Miguel7373
Copy link
Collaborator

Stand 23.1.2025

Ich habe im backend den boolean sowohl dem Quarter gegeben als auch dem Objective damit im Frontend alle regexe ersetzt werden können. Dann habe ich die Frontend-Unit-Tests gefixt. Es failen aber noch zwei E2E-Tests, deshalb muss der Frontend code noch ein wenig angepasst werden.
Es sollten auch noch im Backend Tests gefixt und für die neuen Files geschrieben werden.

@ManuelMoeri ManuelMoeri self-assigned this Jan 24, 2025
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 a pull request may close this issue.

5 participants