continentaldivide / destiny-status

Player lookup tool for Destiny 2
https://destinystatus.net
0 stars 1 forks source link

Missing error handling in useManifestStatus.ts #7

Closed continentaldivide closed 1 year ago

continentaldivide commented 1 year ago

The second useEffect in useManifestStatus.ts sets an up-to-date cacheManifestVersion before ensuring that the fetchManifest function call has succeeded without errors. Need to do some refactoring to manage fail states for the different pieces of logic happening there.