Skip to content

fix(types): add missing config key enableClaims - @lesmo #810

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

Merged
merged 1 commit into from
Jan 2, 2020

Conversation

lesmo
Copy link
Contributor

@lesmo lesmo commented Dec 22, 2019

Description

Linters would fail when using rffConfig.enableClaims as documented

Check List

If not relevant to pull request, check off as complete

  • All tests passing
    (Nothing changed)
  • Docs updated with any changes or examples if applicable
    (Already documented)
  • Added tests to ensure new feature(s) work properly
    (Not needed)

Relevant Issues

(Apprently, none)

Linters would fail when using rffConfig.enableClaims [as documented](http://react-redux-firebase.com/docs/auth.html#custom-claims)
@codecov
Copy link

codecov bot commented Dec 22, 2019

Codecov Report

Merging #810 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #810   +/-   ##
=======================================
  Coverage   91.43%   91.43%           
=======================================
  Files          29       29           
  Lines         759      759           
=======================================
  Hits          694      694           
  Misses         65       65

@rscotten
Copy link
Contributor

rscotten commented Jan 2, 2020

@lesmo Thanks, I was about to add this.

@prescottprue Please merge.

@prescottprue prescottprue changed the base branch from master to v3.0.7 January 2, 2020 20:38
@prescottprue prescottprue changed the title Added missing config key enableClaims fix(types): add missing config key enableClaims Jan 2, 2020
@prescottprue prescottprue changed the title fix(types): add missing config key enableClaims fix(types): add missing config key enableClaims - @lesmo Jan 2, 2020
@prescottprue prescottprue merged commit 26e088c into prescottprue:v3.0.7 Jan 2, 2020
@prescottprue prescottprue mentioned this pull request Jan 2, 2020
3 tasks
prescottprue added a commit that referenced this pull request Jan 10, 2020
* fix(types): add missing config key `enableClaims` - @lesmo - #810
* fix(types): make `updateAuth` params not required and use correct type for `runTransaction` - @rscotten - #821
* chore(deps): update firebase and other dev dependencies - #808
* chore(deps): update dependencies of examples (thanks @dependabot)
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