-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: standalone components #122
feat: standalone components #122
Conversation
Great work @herefishyfish - I'm thinking we plan to release this with ng 17, what do you think? Since it may introduce a breaking change. |
I believe there are no breaking changes with this. I'm wondering if we can also improve the dialogs as well. I'll take a look into how the CDK dealt with it |
Dang it, didn't make v17. Would love to have this as soon as possible 🙏 |
It seems like the commits are now a bit messy, can you rebase it on main? |
3b7027f
to
5ab15b3
Compare
Sorry about that, should be good now. I'm unsure if we should be removing the NgIf's from the platform filters in favour of the new control flow syntax - maybe I'll raise that as another PR. |
@edusperoni @NathanWalker Bump on this one, excited for it! |
5ab15b3
to
abbe4b0
Compare
PR Checklist
What is the current behavior?
What is the new behavior?
Implements standalone components.