cuappdev / cuappdev/score-android

Refactor ScoreBox to accept ScoreBoxViewState

Ouverte
#84 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Kotlin
Étoiles
3
Forks
0
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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_

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par suivre ScoreBox, ScoreBoxViewState, GameData, mapToPeriodScores et toPeriodScores à travers le view model et le composant. Déterminez si le refactoring complet du view state ou la modification plus limitée de GameData est prévu, puis vérifiez que ScoreBox reçoit le state choisi et que les scores par période continuent de s’afficher correctement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
android, kotlin
Domaine
mobile
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.