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.

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

SpotifyApi.authorizationCodeGrant: add parameter codeVerifier. #205

Merged
merged 2 commits into from
Mar 1, 2024

Conversation

gmpassos
Copy link
Contributor

@gmpassos gmpassos commented Feb 20, 2024

Expose codeVerifier when building AuthorizationCodeGrant from SpotifyApi.authorizationCodeGrant.

Issue: #201

@hayribakici
Copy link
Collaborator

@gmpassos please provide a small description (e.g. relation to an issue)

@gmpassos
Copy link
Contributor Author

FYI: @rinukkusu

Copy link
Owner

@rinukkusu rinukkusu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check the comment - looks good otherwise!

lib/src/spotify_base.dart Show resolved Hide resolved
@gmpassos
Copy link
Contributor Author

Please check the comment - looks good otherwise!

Resolved 👍.

After merging this branch, could you please publish it? I prefer to use the published package in our code.

Afterward, I will submit a PR that brings the code into compliance with lints: ^3.0.0.

Thanks in advance.

Best regards.

@rinukkusu rinukkusu merged commit cef576d into rinukkusu:master Mar 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants