search

PatrickAlphaC / defi-stake-yield-brownie-freecode

53 stars 63 forks source link

fix tokenIsAllowed function #22

Closed mmiguenz closed 2 years ago

mmiguenz commented 2 years ago

Hello Patrick! Following the course and adding some test on my own. I found tokenIsAllowed was returning a transaction instead of the value, what was making my assertion fail. I think tokeIsAllowed should have view modifier. is this okey ?

PatrickAlphaC commented 2 years ago

Whoops! Great call! Thanks for making this!!