[PR #208] [MERGED] fix: update Spotify endpoint and align models with new response #212

New issue

Closed

opened 2026-02-27 19:26:47 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 19:26:47 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Lambada10/SongSync/pull/208
Author: @Paxsenix0
Created: 1/7/2026
Status: ✅ Merged
Merged: 1/7/2026
Merged by: @Lambada10

Base: master ← Head: master

📝 Commits (5)

0c23b7b refactor: use Spotify internal API for track search
8331394 refactor: restructure Spotify models for internal search API
868327c Merge pull request #2 from Paxsenix0/Paxsenix0-patch-1
633eeba fix: I accidentally deleted package declaration
1936391 fix: make extensions field optional in TrackSearchResult

📊 Changes

2 files changed (+154 additions, -97 deletions)

View changed files

📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt (+31 -20)
📝 app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/spotify/SpotifyApi.kt (+123 -77)

📄 Description

Updated API endpoint to pathfinder endpoint
Refactored data models to match new response structure
Fixed optional field handling in deserialization

This changes fixes #207

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Lambada10/SongSync/pull/208 **Author:** [@Paxsenix0](https://github.com/Paxsenix0) **Created:** 1/7/2026 **Status:** ✅ Merged **Merged:** 1/7/2026 **Merged by:** [@Lambada10](https://github.com/Lambada10) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (5) - [`0c23b7b`](https://github.com/Lambada10/SongSync/commit/0c23b7b5361a2d682c3fda82c6688076e916b1fe) refactor: use Spotify internal API for track search - [`8331394`](https://github.com/Lambada10/SongSync/commit/83313948ef5fb5acad49b9b833a4777854ad118c) refactor: restructure Spotify models for internal search API - [`868327c`](https://github.com/Lambada10/SongSync/commit/868327caf4758a40642da0f5871502e82f6914f0) Merge pull request #2 from Paxsenix0/Paxsenix0-patch-1 - [`633eeba`](https://github.com/Lambada10/SongSync/commit/633eebaee0d8fae700efd350ff36b33bc0a6efe4) fix: I accidentally deleted package declaration - [`1936391`](https://github.com/Lambada10/SongSync/commit/1936391812423c2437512f8ba37601dcd82f9f23) fix: make extensions field optional in TrackSearchResult ### 📊 Changes **2 files changed** (+154 additions, -97 deletions) <details> <summary>View changed files</summary> 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/spotify/SpotifyAPI.kt` (+31 -20) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/spotify/SpotifyApi.kt` (+123 -77) </details> ### 📄 Description - Updated API endpoint to pathfinder endpoint - Refactored data models to match new response structure - Fixed optional field handling in deserialization This changes fixes #207 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>