-
-
Notifications
You must be signed in to change notification settings - Fork 555
feat(auth): add custom claims #741
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
Conversation
* fix(docs): make examples of HOCs more consistent * fix(examples): update react-native example to use expo * fix(examples): update example deps
From comment here: prescottprue#344 (comment)
it is not true by default
…th to.be.undefined and to.equal(true) with to.be.true
@joerex It looks like you forgot to (re)base your commits on the next branch. There are loads of commits included that aren't yours and 163 changed files in total. |
@joerex your changes look great, nice work! If we point it to |
Used |
* feat(hooks): hook rework to match existing HOC - #734 - @illuminist * fix(hooks): remove create functions (`createUseFirestore`, `createWithFirestore`, `createUseFirebase`, `createWithFirebase`) since store selection is not necessary * feat(auth): add custom claims - #741 - @joerex * fix(types): changed extended firebase instance to function - #743 - @rscotten * fix(types): switch `typeof Firebase` to `any` (prevents issue with passing some version of Firebase JS SDK) * fix(examples): update material and typescript examples
Also added to a PR for v2.4.0 so it can go into latest. Thanks again @joerex |
Description
Firebase has a secure way of identifying and making claims about users with custom claims. This is a good way to provide roles for users.
This PR offers an optional configuration to fetch custom claims with the profile and updates the docs to describe how to do it.
Check List
If not relevant to pull request, check off as complete
Relevant Issues