-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[SDL 0109] SetAudioStreamingIndicator RPC #108
[SDL 0109] SetAudioStreamingIndicator RPC #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kshala-ford I've left a couple of comments. Can you please review and address accordingly? Let me know of any questions. Thanks!
MOBILE_API.xml
Outdated
Default playback indicator. | ||
By default the playback indicator should be PLAY_PAUSE when: | ||
- the media app is newly registered on the head unit (after RegisterAppInterface) | ||
- the media app was closed by the user (App enteres HMI_NONE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should say "enters `HMI_NONE" (removing extra "e" and adding code blocks around "HMI_NONE")
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jacobkeeler can you please fix the typo in the readme?
1b4abbb
to
3e9a1a4
Compare
This PR implements #67 according to the SDL proposal 0109
This PR is ready for review.
Risk
This PR makes minor API changes.,
CLA