-
-
Notifications
You must be signed in to change notification settings - Fork 225
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(tauri): http jellyfin server address #2050
Conversation
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.
You don't need to use setup to create a new window, you can still use the older window created in tauri.conf.json, also you should disable the localhost server when in dev mode since vite already does that
Updated, also added save/restore of window state which is probably more convenient |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
bc856ac
to
7336bfd
Compare
Tauri merged a new option to enable http for windows. This supersedes the localhost plugin. tauri-apps/tauri#7645 |
@endrl That's great news! As soon as I see a Tauri update from renovate I'll merge so you can update this PR accordingly :) |
Added a patch which points to WIP Tauri 1.5. If you want to wait for Tauri 1.5 release, we can close this PR - no changes required :) |
I will have to check if their fix works on Linux before merging it. |
Oh, already released. That's fast. I installed the deb and i noticed:
So we are back at: #2049 |
@endrl But it's the original issue that ti's PR was addressing solved? |
Oh, there is missing flag. Now it's working. You can merge this and update to tauri 1.5 |
6d41b29
to
cfd3d82
Compare
0130be2
to
9dae9fa
Compare
@endrl The checks are still failing for some reason |
@endrl Oops, good catch! Let's rerun checks after the renovate PR. |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Cloudflare Pages deployment
|
Disable https context.
Closes: #2009