Open alexcoaton opened 2 years ago
:)
Reviewed code and ran test and it looks great.
@GracieHayward and @karlbayly You need to open a pull request before this can get a tick from a peer and be reviewed by a teacher. Thank you!
@EliseWei Thank u! https://github.com/pohutukawa-2022/gardenz/pull/57
There are over 150 uses of
.then()
in the codebase at the time of this issue creation. The objective is to eventually get them all converted to a single consistent style withasync...await
, instead, but it would be impractical to do them all at once.Note that there is one exception where async / await can be problematic - inside useEffect() hook calls. Read this blog post to find out how to handle it correctly.
Considering the above exception, find uses of .then() in the following files and replace them with async / await-style promise handling.