Skip to content
This repository was archived by the owner on Mar 17, 2025. It is now read-only.

Report error when streaming loses connection. #179

Closed

Conversation

ed7coyne
Copy link
Collaborator

@ed7coyne ed7coyne commented Jun 1, 2016

Fixes #177.

@@ -18,8 +18,7 @@ FIREBASE_DIR=..
GTEST_DIR=googletest/googletest
ARDUINOJSON_DIR=../src/third-party/arduino-json-5.3

FIREBASE_SRCS=${FIREBASE_DIR}/src/FirebaseObject.cpp\
${FIREBASE_DIR}/src/FirebaseObject.h
FIREBASE_SRCS=${FIREBASE_DIR}/src/FirebaseObject.cpp
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you have the .h here when you run "make clean" it deletes it. We also don't need it.

Copy link
Contributor

@proppy proppy Jun 26, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ouch, maybe we should just have a _DEPS

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we are fine just leaving it out, it works this way.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yep, the only gotcha is that it won't force to recompile if the .h changes.

@yunanhelmy
Copy link

yunanhelmy commented Aug 29, 2016

Hi @ed7coyne,

When device is losing wifi (wifi turned off) I got an exception. Is this because your branch not yet merged?

@proppy proppy added the arduino label Sep 26, 2016
@proppy
Copy link
Contributor

proppy commented Oct 4, 2016

@ed7coyne can you rebase and see if the build still fails?

@SpencerCornish
Copy link

@proppy @ed7coyne Any status on this? it's pretty stale, but I'd love to use this. If it is I'll do some testing and see if I can't pick it up myself.

@proppy
Copy link
Contributor

proppy commented Jun 5, 2018

@SpencerCornish I wonder if this will be fixed by #330 ? /cc @kotl

@proppy
Copy link
Contributor

proppy commented Jun 15, 2018

@kotl is this included in your recent changes?

kotl added a commit to kotl/firebase-arduino that referenced this pull request Jun 21, 2018
@kotl
Copy link
Collaborator

kotl commented Jun 21, 2018

It now is included in this pull request:
#356

@proppy
Copy link
Contributor

proppy commented Jun 21, 2018

Closing since this is now merged.

@proppy proppy closed this Jun 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants