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.

Sign up for GitHub

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

Jump to bottom

use docker image 0.6.03 and change BLE notification to indication #23494

Merged
merged 1 commit into from
Nov 4, 2022
Merged

use docker image 0.6.03 and change BLE notification to indication #23494

merged 1 commit into from
Nov 4, 2022

Conversation

pankore
Copy link
Contributor

@pankore pankore commented Nov 4, 2022 •
edited by andy31415
Loading

Sync Master branch commits #23031 and #23013 to v1.0 branch.

  • use docker image 0.6.03
  • switch from ble notification to indication in ameba BLEManager

@andy31415
Copy link
Contributor

@pankore for future cherrypicks, please use the syntax of #<issue> instead of http:/...../<issue> since that makes the linked PR checker happy (it does not understand http links even if summary formats them identically).

For now I manually updated the summary.

@andy31415 andy31415 merged commit 9898345 into project-chip:v1.0-branch Nov 4, 2022
@pankore
Copy link
Contributor Author

pankore commented Nov 7, 2022

@pankore for future cherrypicks, please use the syntax of #<issue> instead of http:/...../<issue> since that makes the linked PR checker happy (it does not understand http links even if summary formats them identically).

For now I manually updated the summary.

Noted with thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andy31415 andy31415 andy31415 approved these changes

Assignees
No one assigned
Labels
github integrations platform workflows
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pankore @andy31415