123ishaTest/ludiek

Reconsider `amount` for input and output

Offen

#84 geöffnet am 19.08.2025

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (4 Forks)auto 404
good first issuehelp wanted

Repository-Metriken

Stars
 (12 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

#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

Contributor Guide

Research-Richtung
Untersuche die in PR #83 eingeführten Eingabe und Ausgabeklassen, verstehe, wie `amount` derzeit verwendet wird, und implementiere eine `getAmount()` Methode, die auf input.amount zurückfällt oder einen Fehler auslöst. Stelle sicher, dass die Refaktorierung die Abwärtskompatibilität beibehält.
Tech Stack
typescript
Domain
game dev
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55