Skip to content
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 composition api to style-guide #2585

Merged

Conversation

Shinigami92
Copy link
Contributor

@Shinigami92 Shinigami92 commented Nov 30, 2023

Description of Problem

We had issues with some juniors in our company and told them to read the style-guide section about how to handle props and emits between parent and child components,
but then we found out that there are no composition api examples in that section

The juniors only know Vue 3 and don't know the "old" options-api

Proposed Solution

Add composition-api style-guide examples

Additional Information

I'm using TypeScript here, please let me know if I should rewrite the examples to plain JavaScript
All docs around the style-guide uses JavaScript, so I used JS as well

Copy link

netlify bot commented Nov 30, 2023

Deploy Preview for vuejs ready!

Name Link
🔨 Latest commit 640b268
🔍 Latest deploy log https://app.netlify.com/sites/vuejs/deploys/6568a0c281366f0008e74c66
😎 Deploy Preview https://deploy-preview-2585--vuejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Shinigami92 Shinigami92 force-pushed the docs-add-composition-style-guide branch 4 times, most recently from 20f8297 to 5fc2004 Compare November 30, 2023 14:30
@Shinigami92 Shinigami92 force-pushed the docs-add-composition-style-guide branch from 5fc2004 to 640b268 Compare November 30, 2023 14:48
@Shinigami92 Shinigami92 marked this pull request as ready for review November 30, 2023 14:50
Copy link
Member

@NataliaTepluhina NataliaTepluhina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Shinigami92 thank you for the great contribution! Looks great to me, approved 👍🏻

@NataliaTepluhina NataliaTepluhina merged commit 8dc0a33 into vuejs:main Nov 30, 2023
4 checks passed
@Shinigami92 Shinigami92 deleted the docs-add-composition-style-guide branch November 30, 2023 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants