[Accepted with Revisions] SDL 0213 Revisions - rc-radio-climate-parameter-update

[theresalech]

Hello SDL community,

The review of "Revision 0213-rc-radio-climate-parameter-update" begins now and runs through June 18, 2019. This will be a review of proposed revisions to a previously accepted but not yet implemented proposal, SDL 0213.

The pull request outlining the revisions under review is available here:

#757

Reviews are an important part of the SDL evolution process. All reviews should be sent to the associated Github issue at:

#761

What goes into a review?

The goal of the review process is to improve the proposal under review through constructive criticism and, eventually, determine the direction of SDL. When writing your review, here are some questions you might want to answer in your review:

• Is the problem being addressed significant enough to warrant a change to SDL?
• Does this proposal fit well with the feel and direction of SDL?
• If you have used competitors with a similar feature, how do you feel that this proposal compares to those?
• How much effort did you put into your review? A glance, a quick reading, or an in-depth study?
  Please state explicitly whether you believe that the proposal should be accepted into SDL.

More information about the SDL evolution process is available at

https://github.com/smartdevicelink/sdl_evolution/blob/master/process.md

Thank you,
Theresa Lech

Program Manager - Livio
theresa@livio.io

[joeljfischer]

1. Does availableHDsAvailable need a history tag for the pre-deprecation version?

[Jack-Byrne]

1. Thank you for pointing that out.

I would like to change the RPC availableHDsAvailable entry to:

        <param name="availableHDsAvailable" type="Boolean" mandatory="false" deprecated="true" since="5.x">
            <description>
                Availability of the getting the number of available HD channels.
                True: Available, False: Not Available, Not present: Not Available.
            </description>
            <history>
                <param name="availableHDsAvailable" type="Boolean" mandatory="false" since="4.5" until="5.x"/>
            </history>
        </param>

[theresalech]

The Steering Committee voted to accept these changes, with the revision noted in this comment. The Steering Committee also voted to keep this feature in the planned October releases, thereby making the Core and RPC Spec releases major version changes (6.0).

[theresalech]

@JackLivio please advise when PR #757 has been updated to reflect the agreed upon revisions. I'll then merge so that the proposal .md file for SDL 0213 incorporates these changes, and update the implementation issues for SDL 0213 accordingly.

[Jack-Byrne]

@theresalech Accepted revisions have been updated.

[theresalech]

Implementation issues have been updated to reflect accepted revisions:

• RPC
• Core
• iOS
• Java Suite