danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Fix discovery modal footer stuck on 'Discovering...' after completion

Browse source 
The `complete` flag from polling responses was never forwarded into the
transformed status object passed to `updateYouTubeDiscoveryModal`, so
the `if (status.complete)` block that swaps the footer from the
'Discovering...' spinner to the Sync / Download Missing buttons never
fired. Fixed for all three affected sources: Tidal, Deezer, and Spotify
Public — both the WebSocket and HTTP polling paths for each.
This commit is contained in:
Broque Thomas 2026-04-15 15:26:52 -07:00
parent 0cae485cc3
commit 09e08831f9
1 changed files with 6 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
webui/static/script.js
View file

@ -26143,6 +26143,7 @@ function startTidalDiscoveryPolling(fakeUrlHash, playlistId) {
// Transform to YouTube modal format
const transformed = {
progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
complete: data.complete,
results: (data.results || []).map((r, i) => {
const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
return {
@ -26197,6 +26198,7 @@ function startTidalDiscoveryPolling(fakeUrlHash, playlistId) {
progress: status.progress,
spotify_matches: status.spotify_matches,
spotify_total: status.spotify_total,
complete: status.complete,
results: status.results.map((result, index) => {
const isFound = result.status === 'found' ||
result.status === '✅ Found' ||
@ -27586,6 +27588,7 @@ function startDeezerDiscoveryPolling(fakeUrlHash, playlistId) {
}
const transformed = {
progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
complete: data.complete,
results: (data.results || []).map((r, i) => {
const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
return {
@ -27639,6 +27642,7 @@ function startDeezerDiscoveryPolling(fakeUrlHash, playlistId) {
progress: status.progress,
spotify_matches: status.spotify_matches,
spotify_total: status.spotify_total,
complete: status.complete,
results: status.results.map((result, index) => {
const isFound = result.status === 'found' ||
result.status === '✅ Found' ||
@ -31468,6 +31472,7 @@ function startSpotifyPublicDiscoveryPolling(fakeUrlHash, urlHash) {
}
const transformed = {
progress: data.progress, spotify_matches: data.spotify_matches, spotify_total: data.spotify_total,
complete: data.complete,
results: (data.results || []).map((r, i) => {
const isFound = r.status === 'found' || r.status === '✅ Found' || r.status_class === 'found' || r.spotify_data || r.spotify_track;
return {
@ -31521,6 +31526,7 @@ function startSpotifyPublicDiscoveryPolling(fakeUrlHash, urlHash) {
progress: status.progress,
spotify_matches: status.spotify_matches,
spotify_total: status.spotify_total,
complete: status.complete,
results: status.results.map((result, index) => {
const isFound = result.status === 'found' ||
result.status === '✅ Found' ||