ramsayleung / rspotify

Spotify Web API SDK implemented on Rust

MIT License

639 stars 123 forks source link

Add token fetch callback #406

Closed PaulOlteanu closed 1 year ago

PaulOlteanu commented 1 year ago

Description

Please include a summary of the change and which issue is fixed

Motivation and Context

400

Please also include relevant motivation and context.

Dependencies

List any dependencies that are required for this change.

Type of change

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

How has this been tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Please also list any relevant details for your test configuration

[ ] Test A: xxx
[ ] Test B: xxx

Is this change properly documented?

Please make sure you've properly documented the changes you're making.

Don't forget to add an entry to the CHANGELOG if necessary (new features, breaking changes, relevant internal improvements).

PaulOlteanu commented 1 year ago

The place the callback gets called needs to be figured out a bit - I think it should run on the first token fetch, not just on refresh

ramsayleung commented 1 year ago

Closing over #418