[PR #88] [MERGED] Add Musixmatch API Integration #184

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

📋 Pull Request Information

Original PR: https://github.com/Lambada10/SongSync/pull/88
Author: @kerollosy
Created: 10/3/2024
Status: Merged
Merged: 10/15/2024
Merged by: @Lambada10

Base: masterHead: musixmatch


📝 Commits (9)

  • b23dfd2 Add support for Musixmatch lyrics provider
  • d5a28f5 TODO: Handle Musixmatch provider selected
  • c5423b8 Added musixmatchID to track Musixmatch song ID
  • 7c30117 Add musixmatchID field to SongInfo model
  • 656b2d5 Add MusixmatchAPI class for searching and retrieving synced lyrics
  • 83fffdf fix: correct data type for Musixmatch album_id in search response
  • b11d2fc refactor: rename fields in Musixmatch response models for consistency
  • 59429c0 fix: correct Musixmatch API usage in LyricsProviderService
  • 6522e63 feat: add separate fields for synced and unsynced lyrics to Musixmatch API response

📊 Changes

5 files changed (+111 additions, -1 deletions)

View changed files

📝 app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/LyricsProviderService.kt (+9 -0)
app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt (+72 -0)
📝 app/src/main/java/pl/lambada/songsync/domain/model/SongInfo.kt (+1 -0)
app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/others/Musixmatch.kt (+27 -0)
📝 app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt (+2 -1)

📄 Description

This PR adds support for Musixmatch as a lyrics provider, allowing synced lyrics to be retrieved from the Musixmatch API.

Musixmatch is the best lyrics provider afaik
I implemented my own server to filter the Musixmatch output, ensuring compatibility with SongSync's format

I learned kotlin just to make this PR

🔄 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/88 **Author:** [@kerollosy](https://github.com/kerollosy) **Created:** 10/3/2024 **Status:** ✅ Merged **Merged:** 10/15/2024 **Merged by:** [@Lambada10](https://github.com/Lambada10) **Base:** `master` ← **Head:** `musixmatch` --- ### 📝 Commits (9) - [`b23dfd2`](https://github.com/Lambada10/SongSync/commit/b23dfd2ea60a3f728ca99d7a5c4cc2c0e1430ee1) Add support for Musixmatch lyrics provider - [`d5a28f5`](https://github.com/Lambada10/SongSync/commit/d5a28f5415a2d4885da60d7a8fa83d43473d99db) TODO: Handle Musixmatch provider selected - [`c5423b8`](https://github.com/Lambada10/SongSync/commit/c5423b81f25e9c32519ac8679a2c761d24b9252a) Added `musixmatchID` to track Musixmatch song ID - [`7c30117`](https://github.com/Lambada10/SongSync/commit/7c301179eef7bd8e7c3c337030ba95b331e2bdb1) Add musixmatchID field to SongInfo model - [`656b2d5`](https://github.com/Lambada10/SongSync/commit/656b2d53ebcec401d1ad87a590b702df4e849693) Add MusixmatchAPI class for searching and retrieving synced lyrics - [`83fffdf`](https://github.com/Lambada10/SongSync/commit/83fffdfc91453109b35e3065c54de761650bf0da) fix: correct data type for Musixmatch album_id in search response - [`b11d2fc`](https://github.com/Lambada10/SongSync/commit/b11d2fccb19a424c92141cd592bf207ba2b1c514) refactor: rename fields in Musixmatch response models for consistency - [`59429c0`](https://github.com/Lambada10/SongSync/commit/59429c0c8190b19d830d7334701be1e92364069e) fix: correct Musixmatch API usage in LyricsProviderService - [`6522e63`](https://github.com/Lambada10/SongSync/commit/6522e6305d7be46fef663f17ede8b03ec26d67a9) feat: add separate fields for synced and unsynced lyrics to Musixmatch API response ### 📊 Changes **5 files changed** (+111 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/LyricsProviderService.kt` (+9 -0) ➕ `app/src/main/java/pl/lambada/songsync/data/remote/lyrics_providers/others/MusixmatchAPI.kt` (+72 -0) 📝 `app/src/main/java/pl/lambada/songsync/domain/model/SongInfo.kt` (+1 -0) ➕ `app/src/main/java/pl/lambada/songsync/domain/model/lyrics_providers/others/Musixmatch.kt` (+27 -0) 📝 `app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt` (+2 -1) </details> ### 📄 Description This PR adds support for Musixmatch as a lyrics provider, allowing synced lyrics to be retrieved from the Musixmatch API. Musixmatch is the best lyrics provider afaik I implemented my own server to filter the Musixmatch output, ensuring compatibility with SongSync's format ###### I learned kotlin just to make this PR --- <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:41 +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#184
No description provided.