123ishaTest/ludiek

Reconsider `amount` for input and output

Aperta

#84 aperta il 19 ago 2025

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)TypeScript (4 fork)auto 404
good first issuehelp wanted

Metriche repository

Star
 (12 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

#83 introduced input and output. While very useful, the amount property can sometimes be awkward as it doesn't make sense for all resources (e.g. AchievementOutput)

Consider refactoring it to also implement a getAmount() which defaults to input.amount if it's available and if not errors. Basically ensure that one is implemented. Can also already be done at engine construction

Guida contributor

Direzione di ricerca
Esamina le classi input e output introdotte nella PR #83, comprendi come viene attualmente utilizzato `amount` e implementa un metodo `getAmount()` che ricade su input.amount o genera un errore. Assicurati che il refactoring mantenga la compatibilità con le versioni precedenti.
Tech stack
typescript
Dominio
game dev
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55