Preserving local web-ui, added --force-update-gui option to force checkout a new copy of web ui #2117
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Title
Preserving local web-ui, added --force-update-gui option to force checkout a new copy of web ui
Description
Problem
spotdl web --port <port_num>
command downloads the web-ui code from Github and runs it on the specified port. However, the compiled web-ui always points to port 8080, causing API failures on local. This requires a manual port change in the local web-ui repo which gets refreshed on every web run.Solution
Checking if web-ui folder already exists on local before pulling in a fresh checkout.
Added
--force-update-gui
option to force pull the ui repoRelated Issue
Motivation and Context
Helps me run the web ui on a different port without having to manually change the port numbers on every run
How Has This Been Tested?
Since this is such a small change, I just ran some manual tests and checked compilation still worked.
Screenshots (if appropriate)
Types of Changes
Checklist