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

Support for Environmental Service (temperature, pressure and humidity) #75

Merged
merged 3 commits into from
Nov 2, 2015

Conversation

crespum
Copy link
Contributor

@crespum crespum commented Oct 2, 2015

I've added support for another Bluetooth adopted service. It has more characteristics but I've only implemented 3 of them.

@miklis
Copy link

miklis commented Oct 2, 2015

Automatic CI verification build not done, please verify manually.

@rgrover
Copy link
Contributor

rgrover commented Oct 5, 2015

@xcrespo: many thanks for contributing. we'll get around to reviewing this pull request very shortly.

@crespum
Copy link
Contributor Author

crespum commented Oct 5, 2015

I've also updated the Link Loss Service because it was using some deprecated methods and the call to the onDisconnection callback was not supported anymore

@@ -86,7 +87,7 @@ class LinkLossService {
}
}

void onDisconnectionFilter(void) {
void onDisconnectionFilter(const Gap::DisconnectionCallbackParams_t *params) {
Copy link
Contributor

Choose a reason for hiding this comment

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

thanks for taking care to update this.

@rgrover rgrover merged commit 13a9c19 into ARMmbed:master Nov 2, 2015
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.

3 participants