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.
WHY are these changes introduced?
Starting down the path of removing withAppProvider usage into the new world of hooks.
Part of #1995
WHAT is this pull request doing?
Converts Avatar to use hooks.
Reworks the logic for how we handle falling back to using initials if an image fails to load so it relies on a single
status
state that can be either "pending, errored or loading" rather than twohasError
andhasLoading
states.Use react-testing in some Avatar tests as Enzyme doesn't seem to accurately reflect props chagnes for functional components.
How to 🎩
Play with the following to investigate how the onError fallback to initials logic behaves:
Copy-paste this code in
playground/Playground.tsx
:🎩 checklist
README.md
with documentation changes