Skip to content

Improve reset and allow waiting for data to be received #361

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

Closed
wants to merge 25 commits into from

Conversation

posva
Copy link
Member

@posva posva commented Aug 29, 2019

Closes #83

The new wait option should be important for SSR
Documentation for wait option: https://deploy-preview-361--vuefire.netlify.com/api/vuefire.html#options

If you need this, could you please test it?. It has been released under the next tag on npm, you can install it by doing npm i vuefire@next or yarn add vuefire@next (or vuexfire)
The documentation should be enough to understand how the wait option works, it's also used in the examples for vuefire and vuexfire. Thanks 🙏

@codecov-io
Copy link

codecov-io commented Aug 29, 2019

Codecov Report

Merging #361 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #361      +/-   ##
==========================================
+ Coverage   99.72%   99.73%   +<.01%     
==========================================
  Files          14       14              
  Lines         368      373       +5     
  Branches       64       71       +7     
==========================================
+ Hits          367      372       +5     
  Misses          1        1
Impacted Files Coverage Δ
packages/@posva/vuefire-core/src/shared.ts 100% <ø> (ø) ⬆️
packages/vuexfire/src/firestore.ts 100% <100%> (ø) ⬆️
...ackages/@posva/vuefire-core/src/firestore/index.ts 100% <100%> (ø) ⬆️
packages/vuefire/src/rtdb.ts 100% <100%> (ø) ⬆️
packages/@posva/vuefire-core/src/rtdb/index.ts 100% <100%> (ø) ⬆️
packages/vuexfire/src/rtdb.ts 100% <100%> (ø) ⬆️
packages/vuefire/src/firestore.ts 100% <100%> (ø) ⬆️
packages/@posva/vuefire-core/src/index.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b3116f...cd6d0f6. Read the comment docs.

 - @posva/vuefire-core@2.2.0-alpha.0
 - @posva/vuefire-test-helpers@1.3.0-alpha.0
 - documentation@2.1.0-alpha.0
 - vuefire@2.2.0-alpha.0
 - vuexfire@3.2.0-alpha.0
@posva
Copy link
Member Author

posva commented Oct 10, 2019

merged locally

@posva posva closed this Oct 10, 2019
@posva posva deleted the feat/wait branch October 10, 2019 07:44
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.

Wait for arrays or document to be fetched when binding over an existing ref
2 participants