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

fix: support setting authStrategy using .withDefaults({ authStrategy }) #21

Merged
merged 16 commits into from
Sep 14, 2022

Conversation

gr2m
Copy link
Contributor

@gr2m gr2m commented Aug 13, 2021

closes #20

@gr2m gr2m added Type: Bug Something isn't working as documented typescript labels Aug 13, 2021
@gr2m gr2m force-pushed the 20/withDefaults-authStrategy branch from c6fff1d to 70ceeba Compare August 15, 2021 22:20
@gr2m gr2m force-pushed the 20/withDefaults-authStrategy branch from 53647f8 to 1374eac Compare September 9, 2022 22:25
@gr2m gr2m force-pushed the 20/withDefaults-authStrategy branch from 781733c to d46e4a1 Compare September 13, 2022 20:57
@gr2m gr2m force-pushed the 20/withDefaults-authStrategy branch from 4a9741f to 742a365 Compare September 14, 2022 05:35
@gr2m gr2m force-pushed the 20/withDefaults-authStrategy branch from b693e84 to 196229e Compare September 14, 2022 17:31
@gr2m
Copy link
Contributor Author

gr2m commented Sep 14, 2022

I got it as close as I could, unfortunately not fully working :( I'll make a follow up issue and add the limitation to a "known issues" section in the README

@gr2m gr2m merged commit af7fdec into main Sep 14, 2022
@gr2m gr2m deleted the 20/withDefaults-authStrategy branch September 14, 2022 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Something isn't working as documented typescript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make the authStrategy parameter work with the static .withDefaults() method
1 participant