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

Add default value to defaultAuthOptions useWizard param #5647

Merged
merged 1 commit into from May 2, 2023

Conversation

mnholtz
Copy link
Collaborator

@mnholtz mnholtz commented May 2, 2023

What does this PR do?

Checklist

@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Merging #5647 (699f35b) into main (11e2f19) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #5647   +/-   ##
=======================================
  Coverage   64.30%   64.30%           
=======================================
  Files        1032     1032           
  Lines       32439    32440    +1     
  Branches     6162     6163    +1     
=======================================
+ Hits        20859    20860    +1     
  Misses      11580    11580           
Impacted Files Coverage Δ
src/activation/useWizard.ts 95.83% <100.00%> (+0.08%) ⬆️

@github-actions
Copy link

github-actions bot commented May 2, 2023

No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack.

Do not edit this comment manually.

@twschiller twschiller added this to the 1.7.26 milestone May 2, 2023
@mnholtz mnholtz merged commit e8deb73 into main May 2, 2023
14 checks passed
@mnholtz mnholtz deleted the bugfix/activate_wizard_with_services branch May 2, 2023 18:00
twschiller pushed a commit that referenced this pull request May 2, 2023
Co-authored-by: Misha Holtz <mishmish@Mishas-MacBook-Pro.local>
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.

None yet

3 participants