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

Keyresults Duplizieren #614

Closed
10 tasks done
peggimann opened this issue Nov 24, 2023 · 4 comments · Fixed by #1216
Closed
10 tasks done

Keyresults Duplizieren #614

peggimann opened this issue Nov 24, 2023 · 4 comments · Fixed by #1216
Assignees
Labels
request New feature request

Comments

@peggimann
Copy link
Collaborator

peggimann commented Nov 24, 2023

Als User möchte ich beim Duplizieren von Objectives auswählen, welche Key Results mit dupliziert werden.

Anforderungen

  • Im Dialog zum Duplizieren von Objectives kann neu auch ausgewählt werden, welche Key Results mit dupliziert werden
  • Eine Liste der bestehenden KR Titeln ist im Dialog enthalten
  • Aus dieser Liste können mehrere Key Results ausgewählt werden. Default mässig sind alle KR ausgewählt.
  • Der Titel dieses Abschnitts wird eine Frage, "Welche Key Results sollen dupliziert werden?"
  • Mit UX absprechen, wie der Dialog neu aussehen soll.

TODO (Stand 09.12.2024)

  • Failenden Controller-Test fixen
  • Kontrollieren das für sämtlichen neuen Code Tests vorhanden sind.
    • Controller-Tests
    • Service-Tests
    • E2E-Tests
  • Cleanup des codes
  • Review

Akzeptanzkriterien

  • Die Anforderungen sind erfüllt
  • Smoketests wurden gründlich durchgeführt
  • Unittests, Integrationstests, E2ETests sind ergänzt
@peggimann peggimann added the request New feature request label Nov 24, 2023
@peggimann peggimann assigned peggimann and unassigned peggimann Nov 24, 2023
@peggimann peggimann added UX UX is involved and removed request New feature request labels Aug 20, 2024
@peggimann peggimann added this to the Feature Requests BBT milestone Sep 27, 2024
@peggimann peggimann removed the UX UX is involved label Oct 2, 2024
@MasterEvarior MasterEvarior changed the title Objective Duplizieren Keyresults Duplizieren Nov 20, 2024
@kcinay055679 kcinay055679 self-assigned this Nov 21, 2024
@nevio18324 nevio18324 assigned nevio18324 and unassigned nevio18324 Nov 25, 2024
@nevio18324
Copy link
Collaborator

nevio18324 commented Nov 25, 2024

Todo's

  • Feature umsetzten
  • Unit-tests Integration-Tests und E2E Tests erstellen um das Feature voll zu testen
  • Smoketesten

@nevio18324
Copy link
Collaborator

Stand 26.11

Das auswählen der Keyresults beim duplizieren funktioniert, jedoch failen viele Tests noch vor allem im backend unittest

@ManuelMoeri ManuelMoeri linked a pull request Nov 27, 2024 that will close this issue
@ManuelMoeri
Copy link
Collaborator

ManuelMoeri commented Nov 29, 2024

Stand 29.11.2024

Heute habe ich eine offene Bugs beim duplizieren gefixt. Ausserdem habe ich (fast) alle Tests gefixt. Dann ist mir noch aufgefallen das einen komisches behaviour beim E2E-Testen gab. Dort habe ich schlussendlich noch die TestDaten angepasst da sie schlicht und einfach falsch waren. Ein kleines cleanup habe ich Heute auch schon gemacht musste es jedoch reverten da es ein feature kaputt machte.

ToDo nächste Woche

  • Mit UX anschauen das das styling wie erwünscht vorhanden ist
    • Bei der text-truncate Klasse fehlen noch die drei '...' sonst sollte alles gut sein.
  • Cleanup des Codes, zum Teil noch unverständlich
  • Tests im backend laufen jedoch gibt es noch errors -> fixen
  • Unit Test & E2E Tests ergänzen
  • Smoke Testen lassen
  • Mergen

@nevio18324 nevio18324 self-assigned this Dec 3, 2024
@nevio18324
Copy link
Collaborator

Stand 3.12

Ich habe probiert die Objectivetests welche faileten zu reparieren dies ist auch gelungen, dannach wollte ich den shouldReturnIsCreatedWhenObjectWasDuplicated anpassen da sich die DuplcateObjective schnittstelle verändert hat.
Dabei ist folgendes Problem aufgetreten und zwar versendet MockMvc den request mit dem
Content-Type: applicationjson;charset=UTF-8
wodurch ich diese Response erhalten habe

status = 415
Error message = Content-Type 'application/json;charset=UTF-8' is not supported.
Headers = [Accept:"application/json, application/yaml, application/*+json", X-Content-Type-Options:"nosniff", X-XSS-Protection:"0", Cache-Control:"no-cache, no-store, max-age=0, must-revalidate", Pragma:"no-cache", Expires:"0", X-Frame-Options:"DENY"]

Was Ich bereits probiert habe/werde:

  • MockMvc Javadoc durchgelesen ob ich dort etwas zu diesem Behaviour finde dadurch habe ich das gefunden was im Zusammenhang stehen könnte jedoch bezweifle ich esImage
  • Den body einer funktionierenden Frontend anfrage benutzt um zu testen ob das behaviour am body liegt (War nicht der Fall)
  • Stackoverflow frage eröffnet

@ManuelMoeri ManuelMoeri added bug Something isn't working request New feature request and removed bug Something isn't working labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request New feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants