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

Feature/1185 vorarbeiten kommentar anzeigen #1192

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

clean-coder
Copy link
Collaborator

@clean-coder clean-coder commented Nov 21, 2024

Backend

  • CompletedDto sollte ein ObjectiveDto und nicht ein Objective haben. Dies ist nun gefixt.
  • Daraus ergibt sich das Problem, dass die Tests ziemlich unschön werden, weil man ein ObjectiveDto und nicht mehr ein Objective erstellen muss. ObjectiveDto ist ein Record und hat keinen Builder. Dies habe ich nun geändert und für Testing in ch.puzzle.okr.test.dto.builder zwei Builder CompletedDtoBuilder und ObjectiveDtoBuilder implementiert.
  • Zusätzlich habe ich ein cleanup der bestehenden Completed Tests gemacht und die neuen Builder verwendet.

TODOs im Backend für die Lernenden

  • in CompletedController gibt noch kein Endpoint fürs Laden von bestehenden Completed
  • Nicht jedes Objective muss ein Complete haben. Daher sollte der neue Endpoint ein 404 zurückgeben und ein 200 im Erfolgsfall.
  • IT Tests für neuen Endpoint in CompletedControllerIT

Frontend

  • all Service calls mit dem Format /api/v2/completed/ vom ObjectiveService in einen neuen CompletedService zügeln. Es macht Sinn, diese zu trennen, denn es sind verschiedene Dinge (sind im backend in der DB und in den Services und Controllers auch getrennt)
  • Tests sind gefixt

TODOs im Frontend für die Lernenden

  • aktueller Stand bzgl. Completed im Frontend: Completed wir nur verwendet fürs Erstellen und Löschen. Den aktuellen Complete Status für ein Objective kenn das Frontend nicht. Das Rendering von completed Objectives basiert auf dem State im Objective.
  • Im Frontend muss man sich die Daten vom Frontend über den noch nicht existierenden get Endpoint im CompletedController. Es muss sauber behandelt werden, dass für Objectives die kein Completed haben ein 404 zurückkommt.
  • (jest) Tests und Cypress Tests

@clean-coder clean-coder linked an issue Nov 21, 2024 that may be closed by this pull request
@clean-coder clean-coder marked this pull request as ready for review November 21, 2024 13:39
Copy link
Collaborator

@MasterEvarior MasterEvarior left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@peggimann peggimann force-pushed the feature/1185_vorarbeiten_kommentar_anzeigen branch from 1959ef1 to 26b3868 Compare November 22, 2024 09:11
@peggimann peggimann merged commit e9c891d into main Nov 22, 2024
18 checks passed
@peggimann peggimann deleted the feature/1185_vorarbeiten_kommentar_anzeigen branch November 22, 2024 09:20
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.

Vorarbeiten zu "Kommentar anzeigen"
3 participants