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

Revise 0280 - Adding new parameter of requires audio support and bluetoothDeviceAddress #1075

Merged
merged 5 commits into from
Sep 23, 2020

Conversation

Akihiro-Miyazaki
Copy link
Contributor

Introduction

This is an update of SDL-0280.
Add "JavaScript Suite" in Impacted platforms.

Motivation

Add the description of other platforms(such as iOS and JavaScript Suite).

Proposed solution

Add the comment "This flow is only for Android devices.".
Replace the figure of registeringApp_flow.

  1. Add the sequence of RPC OnDeviceStateChanged.
  2. Add the path to the conventional sequence.
  3. Add the MediaStreamingStatus.isAudioOutputAvailable.
    Add BluetoothInfo to OnDeviceStateChanged.

Impact on existing code

Add "JavaScript Suite" to impacted platforms.

@Akihiro-Miyazaki
Copy link
Contributor Author

Hi, @theresalech -san.
This documentation was revised according to this comment.
Could you please review my proposal?
Thank you.

Copy link
Contributor

@theresalech theresalech left a comment

Choose a reason for hiding this comment

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

Hi @Akihiro-Miyazaki, thanks for your PR. I've left one question, and some suggested changes. Please let me know if you have any questions or concerns with addressing my comments. Thank you!

…d-BluetoothDeviceAddress.md

Co-authored-by: theresalech <theresa@livio.io>
Copy link
Contributor

@theresalech theresalech left a comment

Choose a reason for hiding this comment

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

I've suggested one more small change. After this is committed, I'll approve this PR. Thank you!

…d-BluetoothDeviceAddress.md

Co-authored-by: theresalech <theresa@livio.io>
@Akihiro-Miyazaki
Copy link
Contributor Author

Hi, @theresalech -san.
Thank you in advance for your review.
I made a commit all of your suggestion.
Please confirm them.
Thank you.

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.

None yet

2 participants