#684 - Clean up and reorganized Home.jsx, #685 - Added Twitter error handler and display error message
Description
#684 - moved sections into their own React component to reduce cognitive overload. #685 - added an error handler for when a widget fails to load or when all the widgets fails to load
Fixes #684 Fixes #685
Type of change
Please delete options that are not relevant.
- [x] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)
How Has This Been Tested?
Only tested locally
Checklist:
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [x] My changes generate no new warnings
- [x] New and existing unit tests pass locally with my changes
- [x] Any dependent changes have been merged and published in downstream modules
@billy-le is attempting to deploy a commit to a Personal Account owned by @reactplay on Vercel.
@reactplay first needs to authorize it.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Updated |
|---|---|---|---|
| react-play | ✅ Ready (Inspect) | Visit Preview | Oct 14, 2022 at 3:54PM (UTC) |
@atapas @harshsinghatz @koustov hello, I can have someone review these changes?
@atapas Please review it!
@atapas are you on it?
@billy-le could you please resolve the conflict? I will review post that
@koustov okay, it should be updated now
@billy-le thanks. This seems to be a large change. @atapas I feel its better to engage a QA on this PR before moving it next phase