amatt13/BossFight

"Sell()" is a method on PlayerWeapon.

Ouverte

#14 ouverte le 22 janv. 2023

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)C# (1 fork)auto 404
Back Endgood first issue

Métriques du dépôt

Stars
 (1 étoile)
Métriques de merge PR
 (Métriques PR en attente)

Description

This should be moved to the ShopController or mayby a new buy/sell class. The function should be made general such that any item can be sold trough it.

We should also make sure that the player dosen't receive gold if the method fails. Likewise the player<->item relation should not be delted if the method fails.

Guide contributeur

Direction de recherche
Identifiez la méthode Sell() actuelle dans PlayerWeapon, concevez une interface de vente générale, déplacez la dans ShopController, assurez l'intégrité transactionnelle.
Stack technique
csharp
Domaine
game dev
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70