fix: Infinite loading when errors retrieving data #214
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When data is being retrieved from the api it handles the error and displays a notification, but the next render tries to reload the same api call since the value is still
undefined
.This change gives a
null
value for data in the state for the values that are loaded without arguments and adds helper functions that trigger the api calls if the value isundefined
but not if it isnull
. This makes it simpler for the UI to use the state without having to know and handle thenull
state and still allow for lazy loading of the data.fixes #128