cuappdev / cuappdev/score-android

Refactor ScoreBox to accept ScoreBoxViewState

Offen
#84 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Kotlin
Sterne
3
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Ideally `mapToPeriodScores` should be done in the VM, and the `ScoreBox` component as a whole should accept `ScoreBoxViewState` instead of `GameData`. But if you don't want to refactor all of this right now we don't have to. For now, you could just make `toPeriodScores` a function on the `GameData` `data class`.

_Originally posted by @zachseidner1 in https://github.com/cuappdev/score-android/pull/83#discussion_r2370251309_

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, ScoreBox, ScoreBoxViewState, GameData, mapToPeriodScores und toPeriodScores durch das View Model und die Komponente zu verfolgen. Entscheide, ob das vollständige View-State-Refactoring oder die kleinere GameData-Änderung vorgesehen ist, und überprüfe anschließend, dass ScoreBox den ausgewählten State erhält und die Perioden-Scores weiterhin korrekt gerendert werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
android, kotlin
Bereich
mobile
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.