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(docs): cleanup minor typo in useFirestoreConnect.md #949

Merged
merged 2 commits into from
May 31, 2020

Conversation

gregfenton
Copy link
Contributor

QUESTION: this page still indicates that "populate is not supported". Is this still the case for firestore?

Description

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

QUESTION: this page still indicates that "populate is not supported".  Is this still the case for firestore?
@codecov
Copy link

codecov bot commented May 30, 2020

Codecov Report

Merging #949 into master will not change coverage.
The diff coverage is 25.00%.

@@           Coverage Diff           @@
##           master     #949   +/-   ##
=======================================
  Coverage   88.33%   88.33%           
=======================================
  Files          29       29           
  Lines         797      797           
=======================================
  Hits          704      704           
  Misses         93       93           

identical typo fix
@prescottprue prescottprue changed the title Fix minor typo in useFirestoreConnect.md fix(docs): cleanup minor typo in useFirestoreConnect.md May 31, 2020
@prescottprue prescottprue changed the base branch from master to v3.5.1 May 31, 2020 20:45
@prescottprue
Copy link
Owner

Population is supported in Firestore, but just not the full profile population

Agreed that it should be done, I think #915 is for updating the docs

@prescottprue prescottprue merged commit 1966c34 into prescottprue:v3.5.1 May 31, 2020
@prescottprue prescottprue mentioned this pull request May 31, 2020
3 tasks
prescottprue added a commit that referenced this pull request May 31, 2020
* fix(types): fix file blob type for uploadFile and uploadFiles (#950) - @rscotten
* chore(docs): cleanup minor typo in useFirestoreConnect.md (#949) - @gregfenton
* chore(tests): fix typo "merge" to "merges" in reducer test file (#948) - @yukimurasawa
* chore(docs): remove note about populate not being supported in firestore (#915)
* chore(examples): clarify explanation in `watchEvent` example snippet (#910)
* chore(docs): add correct syntax highlighting to example in firestore.md
* chore(build): improve lint command
* chore(build): only require jsdoc comments in source

Co-authored-by: Richard Scotten <rscotten@users.noreply.github.com>
Co-authored-by: gregfenton <greg.fenton@gmail.com>
Co-authored-by: yukimurasawa <yuki.murasawa0826@gmail.com>
Co-authored-by: yukimurasawa <yuki.murasawa.0826@gmail.com>
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