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

Added RSSI info and implementation for readRSSI() into Android aspect of plugin #221

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

howardho
Copy link

Hi,

Initially I was trying to figure out why readRSSI() didn't work on Android. Later I realized it isn't implemented. Yet later on I noticed that RSSI info is only available pretty much during discovery only. Focusing mainly on supporting RSSI info from classic bluetooth devices, I tried to create an implementation of reaRSSI() for Android... however it doesn't seem to be working.

Also added extra info on the JSON data returned. If discovered, it will provide either Classic, BLE or Dual type. Also the major and minor class type is returned.

Howard

RSSI only available during device discovery, so that was added as part
of JSON data returned. Also added type of device and other info for
Android. Tried to get RSSI from connected device and implement the
readRSSI() for Android, but doesn't really work currently.
Update sample return JSON showing new fields in discovery and other data
returning back
@sebj54
Copy link

sebj54 commented Sep 22, 2016

Is it planned to merge this pull request someday? Current version of the plugin does not provide this functionnality 😢

@carlocarlini
Copy link

howardho, i don't really understand if your 2 commits are in your wishlist or is something available to users.
can you please make this clear? is there any updated plugin?

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