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

fix(types) move static firestore interface to where it's implemented #1013

Merged
merged 2 commits into from
Nov 17, 2020

Conversation

zozoens31
Copy link
Contributor

@zozoens31 zozoens31 commented Oct 8, 2020

Description

See the linked issue: This fixes how Typescript describes the static elements of firestore as implemented by Firebase.

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

#1012


This change is Reviewable

@codecov
Copy link

codecov bot commented Oct 8, 2020

Codecov Report

❗ No coverage uploaded for pull request base (v3.8.0@e716897). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             v3.8.0    #1013   +/-   ##
=========================================
  Coverage          ?   88.37%           
=========================================
  Files             ?       29           
  Lines             ?      800           
  Branches          ?        0           
=========================================
  Hits              ?      707           
  Misses            ?       93           
  Partials          ?        0           

@prescottprue
Copy link
Owner

Will get this into the next release after testing it a bit more, thanks for the PR!

@ben4523
Copy link

ben4523 commented Oct 14, 2020

Will get this into the next release after testing it a bit more, thanks for the PR!

Hi, Any update about the release? I really need to access FieldValue static! Actually, we can't ... Thanks

@prescottprue prescottprue changed the base branch from master to v3.8.0 November 17, 2020 23:04
@prescottprue prescottprue merged commit f18c9b6 into prescottprue:v3.8.0 Nov 17, 2020
@prescottprue prescottprue mentioned this pull request Nov 17, 2020
3 tasks
prescottprue added a commit that referenced this pull request Nov 18, 2020
* feat(auth): enable claims without userProfile (#1008) - @rscotten
* fix(types): add arguments to types for onAuthStateChanged (#1018) - @AlexanderArvidsson
* fix(auth): dispatch proper error on reset password (#1016) - @djejaquino
* fix(types) move static firestore interface to where it's implemented (#1013) - @zozoens31
* feat(auth): add applyActionCode method (#994) - @komachi

Co-authored-by: Richard Scotten <[email protected]>
Co-authored-by: Alexander Arvidsson <[email protected]>
Co-authored-by: Davi Aquino <[email protected]>
Co-authored-by: Cyrille Corpet <[email protected]>
Co-authored-by: Anton Nesterov <[email protected]>
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