Skip to content

Conversation

@dzoba
Copy link
Contributor

@dzoba dzoba commented Feb 14, 2020

Description

If you attempt to run the simple example, you will get this error:

image

It's pretty easily solved by the change in this PR.

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

Not applicable. I didn't file an issue for this. Let me know if there needs to be one.

@codecov
Copy link

codecov bot commented Feb 14, 2020

Codecov Report

Merging #866 into master will not change coverage by %.
The diff coverage is n/a.

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

@dzoba
Copy link
Contributor Author

dzoba commented Feb 14, 2020

Looks like there are issues beyond this, since Todos won't come in and can't be added. If you happen to know a quick fix, that would be helpful

image

@prescottprue prescottprue changed the base branch from master to fix-simple-example February 14, 2020 18:08
@prescottprue prescottprue changed the title Simple example won't run without firebase as a dependency. fix(examples): add firebase as a dependency to complete/simple example Feb 14, 2020
@prescottprue
Copy link
Owner

After adding the Firebase sdk I am seeing the todos load and can add one. I'll add support for the database emulator so we can see if it maybe has to do with the project itself

@prescottprue prescottprue merged commit d2adeac into prescottprue:fix-simple-example Feb 14, 2020
prescottprue added a commit that referenced this pull request Feb 14, 2020
#866) - @dzoba

chore(examples): add emulator support to simple example
prescottprue added a commit that referenced this pull request Feb 26, 2020
…ny error (#870) - @macinjoke

* update roles recipe to reflect redux-auth-wrapper v2 changes

* fix(examples): add firebase as a dependency to complete/simple example (#866)

* chore(examples): add emulator support to simple example

* Fix implicit any error

Co-authored-by: Marko Elez <7696111+markoelez@users.noreply.github.com>
Co-authored-by: Scott Prue <prescottprue@users.noreply.github.com>
Co-authored-by: Chris Dzoba <Chris@Dzoba.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