chainventive
commented
10 months ago Merci pour les conseils ! Je m’en vais regarder ce qui a été fait chez Uniswap pour la délégation !
Encore merci pour tout 🙏 !
On Fri 22 Dec 2023 at 15:38, Castagnet Cyril @.***> wrote:
"Contrat : Architecture des contrats complexe mais bien pensée. La factory par copro est fonctionnelle, de meme pour l'utilisation du vrf (meme si je ne suis pas sur qu'un aléa pour determiner un tie break soit necessairement le meilleur moyen). Conseil : eviter assembly dans le nom de fichier sol pour eviter une confusion avec l'idée qu'on verra de l'assembly solidity. Il est bien d'avoir fait des interfaces pour plus facilement communiquer entre contrat. La DAO est très bien gérée. Aucune évidence de sous optimisation ou de faille de sécu. En revanche dans la dao il n'y a pas de delegation, ce qui fait partie des mecanismes important pour comptabiliser le poids de voters (dans les test cela passe par un transfert de token... ce n'est pas le meilleur moyen non non. L'implémentation simple est observable chez uniswap par exemple ici https://github.com/Uniswap/governance/blob/master/contracts/Uni.sol#L38 ) Tests: très bon coverage. Ne pas hésiter a utiliser les hook comme before each pour envoyer les fixture de it identiques dans un describe. Front : meme architecture que projet 3, c'est propre et fait rapidement professionnel. Bon déploiment azure, c'est très bien de reussir a mettre vos nouvelles connaissances dans ce que vous maitrisez deja. Déploiement video et gh dans les regles. Super projet! 9,5/10"
— Reply to this email directly, view it on GitHub https://github.com/chainventive/SyndX/issues/22, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCSVIA7NMCIZQ5CZ73MM6XLYKWLONAVCNFSM6AAAAABA75E6POVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2TIMBRGQ2TAOI . You are receiving this because you are subscribed to this thread.Message ID: @.***>
"Contrat : Architecture des contrats complexe mais bien pensée. La factory par copro est fonctionnelle, de meme pour l'utilisation du vrf (meme si je ne suis pas sur qu'un aléa pour determiner un tie break soit necessairement le meilleur moyen). Conseil : eviter assembly dans le nom de fichier sol pour eviter une confusion avec l'idée qu'on verra de l'assembly solidity. Il est bien d'avoir fait des interfaces pour plus facilement communiquer entre contrat. La DAO est très bien gérée. Aucune évidence de sous optimisation ou de faille de sécu. En revanche dans la dao il n'y a pas de delegation, ce qui fait partie des mecanismes important pour comptabiliser le poids de voters (dans les test cela passe par un transfert de token... ce n'est pas le meilleur moyen non non. L'implémentation simple est observable chez uniswap par exemple ici https://github.com/Uniswap/governance/blob/master/contracts/Uni.sol#L38 ) Tests: très bon coverage. Ne pas hésiter a utiliser les hook comme before each pour envoyer les fixture de it identiques dans un describe. Front : meme architecture que projet 3, c'est propre et fait rapidement professionnel. Bon déploiment azure, c'est très bien de reussir a mettre vos nouvelles connaissances dans ce que vous maitrisez deja. Déploiement video et gh dans les regles. Super projet! 9,5/10"