search

frickerg / Web-Engineering-3

MAS SE (CAS 3) 2024
0 stars 0 forks source link

Feature/appbar gamestate #40

Closed jvanf closed 3 months ago

jvanf commented 3 months ago

enhance game state management, navigation logic and app-bar

jvanf commented 3 months ago

issue #34

jvanf commented 3 months ago

Hey @frickerg Alle Änderungen wurden in feature/limit-request adaptiert. da wo noch nicht 100% klar, gibt es ein entsprechendes Issue dazu. Allerdings sind das nur kleine Sachen.

in feature/limit-requests habe ich ein Utility-Modul (cardUtils) hinzugefügt, in dem jetzt auch das Starten eines spiels ausgelagert ist. Nach diesen Änderungen würde gameUtils eher passen :laughing: ist ein util-Modul okay? ich meinte tobirama hat das mal gesagt, wenn man es nicht übertreibt sei es okay. ansonsten in GameContext?

auf jedenfall ist es jetzt übersichtlicher, redundanz, boilerplate-code etc auch wesentlich weniger vorhanden

Da die änderungen auf feature/limit-requests sind, erstelle ich ein PR von liimit-req auf appbar-gamestate. da solte spiegeln sich deine Vorschläge wirder.

Danke für das Review