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

Feature/Remote control radio and climate parameter update #33

Conversation

IGapchuk
Copy link

@IGapchuk IGapchuk commented Feb 4, 2019

This PR is ready for review.

Risk

This PR makes major API changes.

Summary

SDL remote control shall allow a mobile application to know which HD
sub-channel index has a signal, so that the app can tune the HD radio
to that sub-channel successfully.
According to this motivation, "availableHDs" parameter of RadioControlData
structure marks as deprecated since 5.1 API version and add new parameter
"availableHdChannels" that is a collection of available HD radio channels.
This parameter will be available since 5.1 API version.
Also "availableHDsAvailable" parameter of RadioControlCapabilities
structure marks as deprecated since 5.1 API version and add new parameter
"availableHdChannelsAvailable" that will be available since 5.1 API
version.

SDL remote control shall allow a mobile application to turn the climate
control on or off if the vehicle supports it.
According to this motivation, "climateEnable" parameter add to
ClimateControlData structure. Also "climateEnableAvailable" parameter add
to ClimateControlCapabilities structure.
NOTE: Please, pay attention, API version of deprecated parameters was downgrade for correct manual and auto testing.

Changelog

Breaking Changes

In RadioControlData:

  • Parameter "availableHDs" mark as deprecated since API version 5.1.
  • Parameter "hdChannel" changes his minvalue from 1 to 0.
  • Add new parameter "availableHdChannels" as list of available HD sub-channel indexes, that will be able since API version 5.1.

In RadioControlCapabilities:

  • Add new parameter "availableHdChannelsAvailable" that will able since API version 5.1.

In ClimateControlCapabilities:

  • Add new parameter "climateEnableAvailable", that will be able since API version 5.1;

In ClimateControlData:

  • Add new parameter "climateEnable", that will be able since API version 5.1.

Other parts of delivery:
CI activities: http://opensdl-jenkins.luxoft.com:8080/view/remote_control_radio_and_climate_parameter_update/
HMI Integration guidelines: Ford-Luxoft/sdl_hmi_integration_guidelines#4
SDL requirements: smartdevicelink/sdl_requirements#118
ATF scripts: Ford-Luxoft/sdl_atf_test_scripts#5
HMI: Ford-Luxoft/sdl_hmi#8

CLA

IGapchuk and others added 2 commits February 4, 2019 10:28
SDL remote control shall allow a mobile application to know which HD
sub-channel index has a signal, so that the app can tune the HD radio
to that sub-channel successfully.
According to this motivation, "availableHDs" parameter of RadioControlData
structure marks as deprecated since 5.1 API version and add new parameter
"availableHdChannels" that is a collection of available HD radio channels.
This parameter will be available since 5.1 API version.
Also "availableHDsAvailable" parameter of RadioControlCapabilities
structure marks as deprecated since 5.1 API version and add new parameter
"availableHdChannelsAvailable" that will be available since 5.1 API
version.

SDL remote control shall allow a mobile application to turn the climate
control on or off if the vehicle supports it.
According to this motivation, "climateEnable" parameter add to
ClimateControlData structure. Also "climateEnableAvailable" parameter add
to ClimateControlCapabilities structure.
@IGapchuk IGapchuk changed the title [WIP]Feature/remote control radio and climate parameter update Feature/remote control radio and climate parameter update Feb 4, 2019
@IGapchuk IGapchuk changed the title Feature/remote control radio and climate parameter update Feature/Remote control radio and climate parameter update Feb 4, 2019
@IGapchuk IGapchuk closed this May 23, 2019
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.

3 participants