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(storage): support only firestore when calling upload and writing to db #601

Conversation

dirathea
Copy link
Contributor

Description

When using only firestore and want to write to db to firestore. The checking is incorrect because it only consider firebase.database but not firebase.firestore when write metadata to db

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

#600

@codecov
Copy link

codecov bot commented Dec 31, 2018

Codecov Report

Merging #601 into next will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #601   +/-   ##
=======================================
  Coverage   78.49%   78.49%           
=======================================
  Files          26       26           
  Lines         916      916           
  Branches      166      166           
=======================================
  Hits          719      719           
  Misses        197      197

@prescottprue
Copy link
Owner

Great find! Thanks for making a PR

@prescottprue prescottprue changed the base branch from next to v2.2.6 January 1, 2019 21:31
@prescottprue prescottprue changed the title check firestore when upload & write to db fix(storage): support only firestore when calling upload and writing to db Jan 1, 2019
@prescottprue prescottprue changed the base branch from v2.2.6 to next January 1, 2019 21:32
prescottprue pushed a commit that referenced this pull request Jan 1, 2019
…to db - #601

Co-Authored-By: Aldira Putra Raharja <dirathea@users.noreply.github.com>
@prescottprue prescottprue changed the base branch from next to v3.0.0-alpha.4 January 1, 2019 21:35
@prescottprue prescottprue merged commit a424cbe into prescottprue:v3.0.0-alpha.4 Jan 1, 2019
@prescottprue prescottprue mentioned this pull request Jan 1, 2019
3 tasks
prescottprue added a commit that referenced this pull request Jan 1, 2019
* fix(storage): support only firestore when calling upload and writing - #600, #601
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.

None yet

2 participants