-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Support Angle Bracket syntax and MU layout #124
Conversation
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.
One minor nit-pick, but changes look good overall.
1b62db3
to
d1bb332
Compare
@rwjblue Ready for merging now ( I needed some help with ember-cli/ember-compatibility-helpers#32) Next steps after merging:
|
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.
Couple of minor nits, but looks great
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.
Awesome, thank you!
ember-welcome-page 4.0.0 published 🎉 |
It changes the component copies:
{{welcome-page}}
for Ember < 3.4<WelcomePage />
for Ember >= 3.4and show the corrent location depending if it is Classic or MU layout.
Discussed at ember-cli/ember-cli#8383, we should also: