Skip to content

Jira 870, Change sensortag_button.ino to use localName #465

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

Merged
merged 1 commit into from
Mar 7, 2017

Conversation

SidLeung
Copy link
Contributor

@SidLeung SidLeung commented Mar 4, 2017

Feature added:

  • Since the BLE library now support the Scan
    Response Data processing, update the sketch
    to look for the sensorTag localName instead
    of its specific unit address.

Cod mods:

  1. sensortag_button.ino:
    • Replace MAC address comparison with localName
      string comparison.

Feature added:
  - Since the BLE library now support the Scan
    Response Data processing, update the sketch
    to look for the sensorTag localName instead
    of its specific unit address.

Cod mods:
1. sensortag_button.ino:
   - Replace MAC address comparison with localName
     string comparison.
@SidLeung
Copy link
Contributor Author

SidLeung commented Mar 4, 2017

@russmcinnis , change the senortag_button.ino as per your suggestion. Did update the comment block to make sure the user is aware of the localName may vary. Please test it.

Copy link
Contributor

@russmcinnis russmcinnis left a comment

Choose a reason for hiding this comment

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

Tested sensortag_button.ino with this PR and it is now possible to run without any changes to the sketch for my model of sensor tag. This change is good.

@SidLeung
Copy link
Contributor Author

SidLeung commented Mar 7, 2017

@yashaswini-hanji , please merge the PR to main trunk and close out the ticket.

@ndgbuilder ndgbuilder merged commit 0376547 into arduino:master Mar 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants