-
Notifications
You must be signed in to change notification settings - Fork 10
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
Upgrade frontend to Vue 2.7 #1258
Conversation
`vue-tsc` is aware of `$vuetify` in templates.
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.
Looks good overall, just one main question
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.
Looks good to me, I think #1259 can be merged into this branch. Any reason this PR is still set as draft?
Replace `vue-type-check` with `vue-tsc`
🚀 PR was released in |
Upgrades the frontend to Vue 2.7, which includes the composition API natively. Crucially, this allows us to abandon usage of the vue composition API plugin, which will be going EOL at the end of this year. There's also been some major bugs found in that plugin, so I think it makes sense to do this upgrade before diving into some of the console errors and reactivity bugs we've been seeing in the frontend to at least rule it out as the source.
There's other stuff we can and should do at some point, like switching to
<script setup>
syntax and using vite instead of webpack, since presumably that will ease the burden when we upgrade to Vue 3 someday. But for now, this is the bear minimum needed to move to Vue 2.7.Closes #1189. Requires #1259 to be merged into this branch first.