Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Authentication commands implemented #155

Open
wants to merge 5 commits into
base: next
Choose a base branch
from
Open

Authentication commands implemented #155

wants to merge 5 commits into from

Conversation

tolbxela
Copy link

@tolbxela tolbxela mentioned this pull request Apr 6, 2020
Closed
@rikkit rikkit closed this Jun 26, 2020
@rikkit rikkit reopened this Jun 27, 2020
@rikkit rikkit added this to the 1.3.0 milestone Jun 28, 2020
@tolbxela
Test were fixed
677deaf 
 * TrackUpdateNowPlayingCommand: some values were emptyed
 * GetWeeklyAlbumChartAsync  and GetWeeklyArtistChartAsync were done without parameter "to"
 * UserGetWeeklyTrackChartTests was commented out, since it's bad test now and should be done another way (see my comment in the file)
 * ScrobblesSingle: trackPlayed and scrobbledTrack.TimePlayed are different, when it took to much time to test the solution. The line was commented out.
@kapral kapral mentioned this pull request Jan 7, 2022
Open
@rikkit rikkit changed the base branch from master to next January 4, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.
2 participants
@tolbxela @rikkit