New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrated user-listen api #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this amazing work! 💯
Please remove/delete the files you added by mistake from git.
Listenbrainz/model/musicModel.swift
Outdated
@@ -0,0 +1,135 @@ | |||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please make every struct a separate file in the model folder? Also, Keep the file name the same as the struct name.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Made changes in the music model and updated according to the new JSON response.
Made userName parameter in the API dynamic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome work right here! 💯
Thank you so much for working on this :)
This PR which is having following changes-
1.) Made music model having fields of the user listen api.
2.) Fetched user listens in the listens view from the api.
Note - Please Don't consider the files APICall.swift, APIData.swift and Helper.swift , As I was experimenting things in this branch and unfortunately it also got pushed along with my main code.