prescottprue / react-redux-firebase

Redux bindings for Firebase. Includes React Hooks and Higher Order Components.

https://react-redux-firebase.com

MIT License

2.55k stars 556 forks source link

updating `useFirestoreConnect` examples from usage of `useFirebaseConnect` #853

Closed goleary closed 4 years ago

goleary commented 4 years ago

Description

updating useFirestoreConnect examples to actually use that function instead of useFirebaseConnect

I tested these examples with the latest versions of the libraries.

Check List

If not relevant to pull request, check off as complete

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

Relevant Issues

852

codecov[bot] commented 4 years ago

Codecov Report

Merging #853 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #853   +/-   ##
=======================================
  Coverage   88.76%   88.76%           
=======================================
  Files          29       29           
  Lines         783      783           
=======================================
  Hits          695      695           
  Misses         88       88

prescottprue commented 4 years ago

I used your same example code and just put it directly within the comment in src then regenerated docs. Also added a section to the contributing docs about how to add to the API docs