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

feat: add vue-ts vue3-ts template #650

Merged
merged 6 commits into from
Dec 12, 2022

Conversation

jerrywu001
Copy link
Contributor

@jerrywu001 jerrywu001 commented Dec 1, 2022

What kind of change does this pull request introduce?

image

Feature:

  • add vue-ts vue3-ts template
  • update vue vue3 template

I checked it and it works fine.

@danilowoz In daily development, the VUE3-TS is widely used. It is recommended to merge.

@codesandbox
Copy link

codesandbox bot commented Dec 1, 2022

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 1, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 53e4577:

Sandbox Source
Sandpack Configuration

@jerrywu001
Copy link
Contributor Author

In daily development, the VUE3-TS is widely used. It is recommended to merge.

@danilowoz
Copy link
Member

Hey @jerrywu001, I loved your contribution! But let's keep the minimal necessary files to run a project, so I'd recommend removing the button and counter and leaving only a Hello world. Maybe the word world can come from a variable.

What do you think?

@jerrywu001
Copy link
Contributor Author

Hey @jerrywu001, I loved your contribution! But let's keep the minimal necessary files to run a project, so I'd recommend removing the button and counter and leaving only a Hello world. Maybe the word world can come from a variable.

What do you think?

Ok, let me do some change.

@jerrywu001
Copy link
Contributor Author

Hey @jerrywu001, I loved your contribution! But let's keep the minimal necessary files to run a project, so I'd recommend removing the button and counter and leaving only a Hello world. Maybe the word world can come from a variable.

What do you think?

@danilowoz Done.

@@ -0,0 +1,123 @@
/**
Copy link
Member

Choose a reason for hiding this comment

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

I was wondering, is there any reason why we still maintain vue instead only vue3? At some point in the future we can sunset it and provide only the latest version of Vue

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, we spoke internally, and let's focus on vue3 only. This means we can remove this file and keep the original vue template, and on the next major release, we'll remove the vue template in favor of the vue3.

</script>

<template>
code: `<template>
<h1>Hello {{ msg }}</h1>
</template>
Copy link
Member

Choose a reason for hiding this comment

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

Missing a line break between template and script?

@vercel
Copy link

vercel bot commented Dec 12, 2022

Someone is attempting to deploy a commit to the CodeSandbox Team on Vercel.

A member of the Team first needs to authorize it.

@jerrywu001
Copy link
Contributor Author

The vue-ts template has been removed. @danilowoz

Copy link
Member

@danilowoz danilowoz left a comment

Choose a reason for hiding this comment

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

Thank you!

@danilowoz danilowoz merged commit 035d1b9 into codesandbox:main Dec 12, 2022
@jerrywu001 jerrywu001 deleted the feat/vue-typescript-template branch December 13, 2022 02:57
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