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

fix: remaining type errors #1943

Merged
merged 3 commits into from
Apr 13, 2023
Merged

fix: remaining type errors #1943

merged 3 commits into from
Apr 13, 2023

Conversation

ferferga
Copy link
Member

@ferferga ferferga commented Apr 10, 2023

Locales are fully typed now. Keep track of intlify/vue-i18n#1116 for better TS support for them

The rest of them are waiting for PRs at Vuetify's side, an SDK upgrade and a further investigation of our typed routes setup
Deep type instantiation is also fixed

@jellyfin-bot jellyfin-bot added the vue Pull requests that edit or add Vue files label Apr 10, 2023
Locales are fully typed now. Keep track of intlify/vue-i18n#1116 for better TS support for them

- The rest of them are waiting for PRs at Vuetify's side, an SDK upgrade and a further investigation of our typed routes setup.
- Deep type instantiation is also fixed.
unplugin-vue-router uses a different name generation for its routes, so the names didn't overlap beforehand
We're already pinned on ES2022, so we don't need the TODO comment
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@jellyfin-bot
Copy link

Cloudflare Pages deployment

Latest commit deffb07
Status ✅ Deployed!
Preview URL https://f1302226.jf-vue.pages.dev
Type 🔀 Preview

View build logs
View bot logs

@ferferga ferferga merged commit 16955dd into master Apr 13, 2023
@ferferga ferferga deleted the fix-types branch April 13, 2023 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vue Pull requests that edit or add Vue files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants