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

Add validation ID to BLE #1666

Merged
merged 1 commit into from Apr 16, 2023
Merged

Add validation ID to BLE #1666

merged 1 commit into from Apr 16, 2023

Conversation

ew1abz
Copy link
Contributor

@ew1abz ew1abz commented Apr 16, 2023

Please check if the PR fulfills these requirements

  • The changes have been tested locally (see the attached picture)
  • There are no breaking changes

What kind of change does this PR introduce?

Resolves #1609

What is the current behavior?

Device validation ID is not available via BLE.

What is the new behavior (if this is a feature change)?

The serial number and validation ID are now both exposed via BLE.
Clarifies naming in the source code. The detailed explanation is in the comments for #1609.

photo_2023-04-16_14-32-29

@discip discip enabled auto-merge April 16, 2023 22:08
@discip discip requested a review from Ralim April 16, 2023 22:08
@discip discip merged commit 546ac3a into Ralim:dev Apr 16, 2023
11 checks passed
@Ralim
Copy link
Owner

Ralim commented Apr 16, 2023

Please update to resolve merge conflicts and then happy to merge :)

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.

BLE - need serial number for authenticity checker
3 participants