-
Notifications
You must be signed in to change notification settings - Fork 37
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
App hierarchy #152
App hierarchy #152
Conversation
|
||
expect(() => widget.setRegion('sidebar')).to.throw(/No root app instance available/); | ||
}); | ||
// expect(() => widget.setRegion('sidebar')).to.throw(/No root app instance available/); |
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.
why you didn't remove this?
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.
I moved all the tests of frint v0.x to frint-compat
and kept them unchanged to make sure we can depend on frint-compat
reliably.
If I really had to remove any individual test, I added a NOTE
on top of the commented block explaining why it's not needed any more.
Just to keep a reference.
What's done
Root App ==> Widget
hierarchyRoot App ==> As ==> Many ==> Apps ==> Widget
Future
This PR now makes way for things like: