cuappdev / cuappdev/score-android
Refactor ScoreBox to accept ScoreBoxViewState
- 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