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

chore: bump @jellyfin/client-axios to 10.7.4 #1018

Closed
wants to merge 1 commit into from

Conversation

ferferga
Copy link
Member

I fixed the useSingleParameter option upstream. Fixed package is up in NPM and GitHub Packages with version 10.7.4 (not yet tagged as latest due to a permission issue in CI)

However, there are some fields in the parameters that are like uNKNOWNBASETYPE for some reason. I wonder if it's a generator bug because we're fully ES6 now or something in server that's not compliant with OpenAPI spec.

This update also reduces the bundle size by quite a lot :) (master is 4.3 MB)
image

@jellyfin-bot jellyfin-bot added vue Pull requests that edit or add Vue files frontend:store Pull requests or issues that deal with stores labels Apr 14, 2021
@sonarcloud
Copy link

sonarcloud bot commented Apr 14, 2021

Kudos, SonarCloud 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

@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 86cf1ea
Status: ✅  Deploy successful!
Preview URL: https://1fdb6bfd.jellyfin-vue.pages.dev

View logs

@heyhippari heyhippari added the blocked Something depends on another issue or Pull Request label Apr 21, 2021
@heyhippari
Copy link
Contributor

Marking as blocked due to an upstream bug with the Axios generator in openapi-generator.

OpenAPITools/openapi-generator#9163

@jellyfin-bot jellyfin-bot added the merge conflict Something has merge conflicts label May 12, 2021
@heyhippari
Copy link
Contributor

Closing this, as the client is still not working properly and a new version would be published anyway.

@heyhippari heyhippari closed this Jul 18, 2021
@ferferga ferferga deleted the bump-jellyfin-client-axios branch December 9, 2021 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked Something depends on another issue or Pull Request frontend:store Pull requests or issues that deal with stores merge conflict Something has merge conflicts vue Pull requests that edit or add Vue files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants