-
Notifications
You must be signed in to change notification settings - Fork 24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Loading UI #557
Loading UI #557
Conversation
@unstubbable Code & Tests are ready for review. It would be helpful if you could already start reviewing this part if you can make the time. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great effort with the tests! 👏 Please excuse my whitespace review comments, this repo unfortunately adheres to an undocumented rule where newlines are added before and after multi-line statements. The main comment of my review is the one regarding the logic on componentDidMount
.
@unstubbable @stemey As far as i can tell i handled everything you commentend and what we talked about. (Demo & Docs still missing) |
(force push was no-conflict rebase onto master only) |
Feature to allow for the integrator to show a generic loading UI, while the feature app is loading.
This deprecates the
renderError
prop.fixes #296
fixes #295