Skip to content

Conversation

@ssdns
Copy link
Contributor

@ssdns ssdns commented Dec 14, 2020

Description

My configuration is userProfile: null, enableClaims: true (ref: #1008) and without setup firebase/database.
It throws "firebase.database is not function" error in firebase.logout.

Check List

If not relevant to pull request, check off as complete

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

Relevant Issues

@codecov
Copy link

codecov bot commented Dec 14, 2020

Codecov Report

Merging #1042 (b6423f1) into master (c72de9d) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1042      +/-   ##
==========================================
+ Coverage   88.33%   88.44%   +0.11%     
==========================================
  Files          29       29              
  Lines         797      805       +8     
==========================================
+ Hits          704      712       +8     
  Misses         93       93              

@prescottprue prescottprue changed the base branch from master to v3.9.0 January 4, 2021 22:31
@prescottprue prescottprue merged commit a1dbc74 into prescottprue:v3.9.0 Jan 4, 2021
@prescottprue prescottprue mentioned this pull request Jan 4, 2021
3 tasks
prescottprue added a commit that referenced this pull request Jan 5, 2021
* fix(auth): prevent throw in `logout` when database is not setup (#1042) - @ssdns
* fix(types): fix `reloadAuth` argument type and duplicate of `endBefore` (#1031) - @ MatthewDailey
* fix(types): fix argument type for `login` with credentials (#1045) - @AlexanderArvidsson
* fix(types): add back `FirestoreStatics` to `ExtendedFirestoreInstance` (#1030)
* fix(core): add deprecation message `enableLogging` config option
* chore(docs): add note about required rules for profile settings (#1049)

Co-authored-by: Matthew Dailey <[email protected]>
Co-authored-by: Alexander Arvidsson <[email protected]>
Co-authored-by: shishido <[email protected]>
@ssdns ssdns deleted the logout-without-database branch January 5, 2021 08:48
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