Skip to content

Conversation

@lucasharada
Copy link
Contributor

Description

Add support for Microsoft OAuthProvider using Firebase.

Check List

  • All tests passing
  • Docs updated with any changes or examples if applicable
  • Added tests to ensure new feature(s) work properly

@codecov
Copy link

codecov bot commented Mar 10, 2020

Codecov Report

Merging #880 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #880      +/-   ##
==========================================
+ Coverage   88.77%   88.81%   +0.04%     
==========================================
  Files          29       29              
  Lines         784      787       +3     
==========================================
+ Hits          696      699       +3     
  Misses         88       88

@prescottprue prescottprue changed the base branch from master to v3.2.0 March 21, 2020 16:40
@prescottprue prescottprue changed the title Microsoft support feat(auth): add support for microsoft oauthprovider - @lucasharada Mar 21, 2020
@prescottprue prescottprue merged commit c6ed2b9 into prescottprue:v3.2.0 Mar 21, 2020
@prescottprue prescottprue mentioned this pull request Mar 21, 2020
3 tasks
prescottprue added a commit that referenced this pull request Mar 21, 2020
* feat(auth): add support for microsoft auth provider - @lucasharada (#880)
* fix(types): switch order of schema type arguments - @macinjoke (#875)
* feat(docs): update docs include microsoft.com provider - @lucasharada (#880)
* feat(tests): add test for microsoft.com auth provider - @lucasharada (#880)
* chore(ci): switch to Github Actions for CI in place of TravisCI
* chore(build): add lint-staged for quicker lint checking on pre-commit
* chore(deps): update dev dependencies including babel deps

Co-authored-by: Lucas Harada <[email protected]>
Co-authored-by: macinjoke <[email protected]>
@prescottprue
Copy link
Owner

Thanks for the PR with the tests included! Great work!

Released in v3.2.0

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.

2 participants