search

Cyfrin / foundry-defi-stablecoin-cu

249 stars 117 forks source link

Adding more modularity for the _getUsdValue function #92

Closed reverse-banana closed 3 months ago

reverse-banana commented 3 months ago

Updated the _getUsdValue function flow to work with any token decimal amount returned by a AggregatorV3Interface

Fyi: The refactored code already been tested

PatrickAlphaC commented 3 months ago

Thanks! This is indeed an improvement :)

For the purpose of the curriculum though, I'm going to leave it as such.

reverse-banana commented 3 months ago

Got it np, @PatrickAlphaC wanna take a moment to personally thank you for the course!