Skip to content

Conversation

microbit-carlos
Copy link
Contributor

@microbit-carlos microbit-carlos commented Mar 27, 2024

This service enables datalog access to the apps, to be able to view the logged data without a computer.

This flag enables the service in pairing mode only and should have no effect when running MicroPython.

More info in:

@microbit-carlos
Copy link
Contributor Author

Okay, I've tested the CI builds and can access the log from the iOS app 🎉

@dpgeorge
Copy link
Collaborator

dpgeorge commented Apr 3, 2024

This looks fine to me, I can't think how it would interfere with MicroPython, which only accesses the log through CODAL.

Note: I have not tested this PR.

"DEVICE_STACK_SIZE": 8192,
"MICROBIT_BLE_ENABLED" : 0,
"MICROBIT_BLE_PAIRING_MODE": 1,
"MICROBIT_BLE_UTILITY_SERVICE_PAIRING": 1,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this go in alphabetical order, ie after MICROBIT_BLE_SECURITY_MODE?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I placed it there to be next to the other pairing mode config option, but make sense to move it alphabetically 👍

This service enables datalog access to the apps, to be able to
view the logged data without a computer.

This flag enables the service in pairing mode only and should
have no effect when running MicroPython.
@dpgeorge dpgeorge force-pushed the ble-utility-service branch from 507db84 to f1a4647 Compare April 29, 2024 03:04
@dpgeorge dpgeorge merged commit f1a4647 into master Apr 29, 2024
@dpgeorge dpgeorge deleted the ble-utility-service branch April 29, 2024 03:08
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.

2 participants