loupsteb / lifeestate-project Goto Github PK

"Contrats : Le projet permet notamment d'acheter des parts avec des tokens approuvés. En revanche, les formules ne tiennent pas comptes de la valeur du token. Il aurait été interessant d'utiliser chainlink, un mock, ou bien juste de n'accepter qu'un type de token (admettons le LoupUsd par exemple). En revanche très bonne gestion de la marketplace ! La figure technique de la marketplace NFT est respectée en back.
Tests : Très bon, le coverage semble bon et les scenarios cohérent. Bonne gestion des fixtures. En revanche on pourrait utiliser en plus des hooks before each dans des context pour load la bonne fixture pour les it de ce context.
Front : Le front est un peu cassé a plusieurs endroits. L'affichage est simple mais il est fonctionnel. En revanche, pas d'upload sur ipfs du nft déployé, comme demandé dans la figure technique.
Déploiement, video et GH dans les règles.
C'est un bon travail."