-
Notifications
You must be signed in to change notification settings - Fork 119
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
docs: add composables docs #1062
Conversation
6303c27
to
144c850
Compare
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.
Thanks for that @filipsobol, it's awesome!
Please resolve conflicts.
@Frodigo I resolved the conflict. I also asked @bartoszherba to review certain parts of the docs we discussed. I'll merge it after his approval. |
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.
All is good but useFetch description must be changes to avoid naming this hook as a lifecycle hook, please check https://composition-api.nuxtjs.org/lifecycle/usefetch/
7d84905
to
cf5edd9
Compare
7723f8e
to
7680a23
Compare
This PR explains what composables are (with Magento-specific examples) and creates one page that lists all composables instead of listing each composable in the sidebar.
Preview: