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

BluePromise. missing. #11

Closed
wants to merge 1 commit into from
Closed

BluePromise. missing. #11

wants to merge 1 commit into from

Conversation

nklerk
Copy link
Contributor

@nklerk nklerk commented Apr 28, 2017

I think reject(new Error('COMPONENTNAME_NOT_FOUND')); should have been
BluePromise.reject(new Error('COMPONENTNAME_NOT_FOUND'));

I think reject(new Error('COMPONENTNAME_NOT_FOUND')); should have been 
BluePromise.reject(new Error('COMPONENTNAME_NOT_FOUND'));
@neophob
Copy link
Contributor

neophob commented Apr 29, 2017

Hi, thanks for the report.

check the header of the _findNotificationKey function - there is the reject defined

  return new Promise((resolve, reject) => {

also see travis build. What's the issue you have?

@nklerk
Copy link
Contributor Author

nklerk commented Apr 30, 2017 via email

@neophob
Copy link
Contributor

neophob commented May 2, 2017

@nklerk any more hints here? I guess the main issue you reported will be fixed with #4

Feel free to reopen if there's a misunderstanding.

Thanks.

@neophob neophob closed this May 2, 2017
@nklerk
Copy link
Contributor Author

nklerk commented May 7, 2017

-Start the discoverable device example.
-Add the device that is reachable (unique-device-id-001)
-Add all the shortcuts to the device and see values changing by the example code.
-Stop the example code.
-Remove the discoverable device from the neeo
-Start example script again. You will get the code error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants