Fix `is_playable` always being `None` #260

marioortizmanero · 2021-09-25T19:13:58Z

Description

As #190 explains, is_playable was always None because the market parameter was missing in these endpoints.

I suggest merging #259 first, as I wrote this over that branch by mistake.

Closes #190

None

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

Check out the test I modified. Can be ran with, for example:

$ cargo test --no-default-features --features=client-ureq,ureq-rustls-tls -- --ignored test_current_user_saved_tracks_add

marioortizmanero added 2 commits September 25, 2021 21:11

Fix current_user_saved_{tracks,albums}

d80ab80

Update changelog

fc56139

marioortizmanero linked an issue Sep 25, 2021 that may be closed by this pull request

is_playable field is always Option::None #190

Closed

marioortizmanero force-pushed the is-playable-tests branch from 0b0115d to 7e63404 Compare September 25, 2021 20:47

Fix examples and tests

ef3a000

marioortizmanero force-pushed the is-playable-tests branch from 7e63404 to ef3a000 Compare September 25, 2021 20:55

ramsayleung approved these changes Sep 28, 2021

View reviewed changes

ramsayleung merged commit d0add84 into percent-search Sep 28, 2021

marioortizmanero deleted the is-playable-tests branch September 28, 2021 11:15