Nalochen / jtr-2.0

Apache License 2.0
4 stars 0 forks source link

Add ngrx to project and create tournamentDetails and tournamentOverview State #104

Closed LeanderCS closed 1 month ago

LeanderCS commented 1 month ago

Fixes #102

Ihr braucht euch nur den 1. Commit anschauen... Der 2. besteht nur aus löschen von Tests

github-actions[bot] commented 1 month ago

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL1172479% 
report-only-changed-files is enabled. No files were changed during this commit :)

Backend coverage

Tests Skipped Failures Errors Time
9 0 :zzz: 0 :x: 0 :fire: 0.514s :stopwatch:
github-actions[bot] commented 1 month ago

Frontend coverage

Tests Skipped Failures Errors Time
22 0 :zzz: 0 :x: 0 :fire: 10.451s :stopwatch:
Coverage Report • (96%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files97.266010096.77 
libs/infrastructure/cache/src/lib/decorators93.546010093.54 
   memory-cache.decorator.ts95.237510095.2317
   singleton-getter.decorator.ts9001009011
Backend-Connoisseur commented 1 month ago

Backend: Hab jetzt nichts Relevantes gefunden. Linter war ja auch schon drüber, so wie es für mich aussieht. Frage wäre (vor allem aus Interesse) warum 'ne redis? Macht auf mich nur den Eindruck von 'nem Overkill (Luxusproblem).

Frontend: Hatte Nalo ja schon bzgl. Mehrsprachigkeit gefragt. Das war, soweit ich das mitbekommen habe, offen. Sehe noch Text (Überschriften) in den html-Files, nehme mal an, dass das 'ne Zukunftsbaustelle ist. Hab mir Angular bisher nur mal angeguckt, daher kann ich zur Funktionalität und grafischen Geschichten eher wenig sagen, hab aber viel mit node.js gearbeitet. Auch hier nichts zu beanstanden. Tests scheinen ja auch alle zu gehen.

Alles in allem schöner Code, gefällt mir :)

LeanderCS commented 1 month ago

@Backend-Connoisseur

Backend: Hab jetzt nichts Relevantes gefunden. Linter war ja auch schon drüber, so wie es für mich aussieht. Frage wäre (vor allem aus Interesse) warum 'ne redis? Macht auf mich nur den Eindruck von 'nem Overkill (Luxusproblem).

Frontend: Hatte Nalo ja schon bzgl. Mehrsprachigkeit gefragt. Das war, soweit ich das mitbekommen habe, offen. Sehe noch Text (Überschriften) in den html-Files, nehme mal an, dass das 'ne Zukunftsbaustelle ist. Hab mir Angular bisher nur mal angeguckt, daher kann ich zur Funktionalität und grafischen Geschichten eher wenig sagen, hab aber viel mit node.js gearbeitet. Auch hier nichts zu beanstanden. Tests scheinen ja auch alle zu gehen.

Alles in allem schöner Code, gefällt mir :)

Erst einmal vielen Dank für dein Feedback und deine Zeit.

BE: Noch wird Redis nicht verwendet und die Frage ob es zu implementieren Overkill sein, habe ich mir auch schon gestellt. Bei kommenden Fällen würde ich es jedoch mal einbauen und dann gucken... Im 'Notfall' kann man es ja wieder ausbauen, so groß wird das Projekt (BE-Seitig) ja nicht.

FE: Gegebenenfalls warst du der Auslöser, denn vor ein paar Tagen hatte Nalo ein Ticket zu der Mehrsprachigkeit erstellt und ist tatsächlich gerade dabei es zu bearbeiten: #94 Dementsprechend wurde bisher noch nicht in die Richtung eingebaut.