[PR #201] [MERGED] feat: use Apple Music API for search and update AppleAPI base URL #211

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

📋 Pull Request Information

Original PR: https://github.com/Lambada10/SongSync/pull/201
Author: @Paxsenix0
Created: 11/30/2025
Status: Merged
Merged: 12/20/2025
Merged by: @Lambada10

Base: masterHead: master


📝 Commits (3)

  • 93fac97 fix: unable to search on Apple Music provider
  • d7b4472 oops, i forgot
  • 2ddc693 fix: Do not get apple jwt t token everytime

📊 Changes

5 files changed (+255 additions, -95 deletions)

View changed files

📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/LyricsProviderService.kt (+7 -4)
app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/apple/AppleAPI.kt (+121 -0)
app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/apple/AppleTokenManager.kt (+46 -0)
app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/AppleAPI.kt (+0 -82)
📝 app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/others/Apple.kt (+81 -9)

📄 Description

  • The Apple Music provider now uses the Apple Music API for song search.
  • Added automatic JWT token generation (may cause timeouts for users on slow connections).
  • Updated the AppleAPI base URL to fetch lyrics.

Please review the changes when you have time! and let me know if anything should be adjusted, I'm afraid this changes might slow down the app.


🔄 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/201 **Author:** [@Paxsenix0](https://github.com/Paxsenix0) **Created:** 11/30/2025 **Status:** ✅ Merged **Merged:** 12/20/2025 **Merged by:** [@Lambada10](https://github.com/Lambada10) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (3) - [`93fac97`](https://github.com/Lambada10/SongSync/commit/93fac97696d40d240017cbda8a096053a27dd93a) fix: unable to search on Apple Music provider - [`d7b4472`](https://github.com/Lambada10/SongSync/commit/d7b4472ee86ad6c8237aacf624ed7333f75e0112) oops, i forgot - [`2ddc693`](https://github.com/Lambada10/SongSync/commit/2ddc6937dc0cb11f1086fd009a79149ba5a69c50) fix: Do not get apple jwt t token everytime ### 📊 Changes **5 files changed** (+255 additions, -95 deletions) <details> <summary>View changed files</summary> 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/LyricsProviderService.kt` (+7 -4) ➕ `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/apple/AppleAPI.kt` (+121 -0) ➕ `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/apple/AppleTokenManager.kt` (+46 -0) ➖ `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/AppleAPI.kt` (+0 -82) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/others/Apple.kt` (+81 -9) </details> ### 📄 Description - The Apple Music provider now uses the Apple Music API for song search. - Added automatic JWT token generation (may cause timeouts for users on slow connections). - Updated the AppleAPI base URL to fetch lyrics. Please review the changes when you have time! and let me know if anything should be adjusted, I'm afraid this changes might slow down the app. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-27 19:26:47 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/SongSync#211
No description provided.