123ishaTest/ludiek

Reconsider `amount` for input and output

Aberta

#84 aberto em 19 de ago. de 2025

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)TypeScript (4 forks)auto 404
good first issuehelp wanted

Métricas do repositório

Stars
 (12 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

#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

Guia do colaborador

Direção de pesquisa
Examine as classes de entrada e saída introduzidas no PR #83, entenda como `amount` é usado atualmente e implemente um método `getAmount()` que recai para input.amount ou lança um erro. Garanta que a refatoração mantenha a compatibilidade retroativa.
Pilha de tecnologia
typescript
Domain
game dev
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
TypeScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55