Draft
Conversation
* Initial plan * Remove 20 request classes, their test classes, and test fixtures Co-authored-by: dargmuesli <[email protected]> * Remove deleted request class references and add new library builder methods in SpotifyApi.java Co-authored-by: dargmuesli <[email protected]> * Remove deprecated endpoints and add new /me/library endpoints - Remove 20 deprecated request classes (GET several/top, POST/GET user-specific, PUT/DELETE per-type library) - Remove corresponding 20 test classes and 12 test fixtures - Remove 28 builder methods from SpotifyApi.java - Add SaveToLibraryRequest (PUT /me/library) and RemoveFromLibraryRequest (DELETE /me/library) - Add saveToLibrary() and removeFromLibrary() builder methods to SpotifyApi.java Co-authored-by: dargmuesli <[email protected]> * Remove availableMarkets, externalIds, label, and popularity fields from Album model Co-authored-by: dargmuesli <[email protected]> * Remove albumGroup and availableMarkets fields from AlbumSimplified Co-authored-by: dargmuesli <[email protected]> * Remove followers and popularity fields from Artist model class Co-authored-by: dargmuesli <[email protected]> * Remove availableMarkets, externalIds, linkedFrom, and popularity fields from Track model Remove field declarations, constructor assignments, getter methods, Builder setter methods, and JsonUtil deserialization code for the four fields. Also remove unused imports (CountryCode, ExternalId, TrackLink, Gson) and update toString method. Update GetTrackRequestTest to remove assertions for removed fields. Co-authored-by: dargmuesli <[email protected]> * Remove availableMarkets and linkedFrom fields from TrackSimplified Co-authored-by: dargmuesli <[email protected]> * Remove availableMarkets and publisher fields from Show and ShowSimplified Co-authored-by: dargmuesli <[email protected]> * Remove country, email, followers, and product fields from User model Co-authored-by: dargmuesli <[email protected]> * Rename tracks/track fields to items/item in Playlist, PlaylistSimplified, and PlaylistTrack Co-authored-by: dargmuesli <[email protected]> * Fix test compilation errors after model field removals Co-authored-by: dargmuesli <[email protected]> * Rename playlist JSON fixture fields: tracks→items, track→item Co-authored-by: dargmuesli <[email protected]> * Add tests for SaveToLibraryRequest and RemoveFromLibraryRequest endpoints Co-authored-by: dargmuesli <[email protected]> * Update examples directory for API changes and add compile-examples CI step - Remove 20 example files for deleted endpoints - Add SaveToLibraryExample and RemoveFromLibraryExample - Fix pre-existing SavedEpisode import bug in GetUsersSavedEpisodesExample - Add compile-examples Maven profile to compile examples with project sources - Add CI step to compile examples during builds Co-authored-by: dargmuesli <[email protected]> * Fix Javadoc for renamed playlist fields and add @OverRide to library request execute methods - PlaylistSimplified.setItems(): update @param tracks → items - Playlist.setItems(): update Javadoc wording and @param tracks → items - PlaylistTrack.setItem(): update Javadoc wording and @param track → item - SaveToLibraryRequest.execute(): add @OverRide annotation - RemoveFromLibraryRequest.execute(): add @OverRide annotation Co-authored-by: dargmuesli <[email protected]> * ci: compile examples by default --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: dargmuesli <[email protected]> Co-authored-by: Jonas Thelemann <[email protected]>
…playlist-endpoint Add CreatePlaylist endpoint (POST /v1/me/playlists)
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## main #455 +/- ##
=============================================
- Coverage 66.64% 49.84% -16.80%
+ Complexity 672 582 -90
=============================================
Files 184 243 +59
Lines 7264 8626 +1362
Branches 1151 1054 -97
=============================================
- Hits 4841 4300 -541
- Misses 1520 3526 +2006
+ Partials 903 800 -103 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🚀 Features