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

Fixed the Show RPC's templateConfiguration parameter not getting set #1489

Conversation

NicoleYarroch
Copy link
Contributor

Fixes #1486

This PR is ready for review.

Risk

This PR makes no API changes.

Testing Plan

  • I have verified that I have not introduced new warnings in this PR (or explain why below)
  • I have run the unit tests with this PR
  • I have tested this PR against Core and verified behavior (if applicable, if not applicable, explain why below).

Unit Tests

Added unit tests to the SDLShowSpec

Core Tests

This PR does not need to be tested against SDL Core. I verified that the parameter is now being set in the test cases for SDLShow.

Core version / branch / commit hash / module tested against: n/a
HMI name / version / branch / commit hash / module tested against: n/a

Summary

Fixed the Show RPC's templateConfiguration parameter not getting set.

Changelog

Bug Fixes
  • The SDLShow's templateConfiguration parameter is now set and retrieved correctly.

CLA

@NicoleYarroch NicoleYarroch added the bug A defect in the library label Dec 5, 2019
@NicoleYarroch NicoleYarroch self-assigned this Dec 5, 2019
@joeljfischer joeljfischer added this to In progress in v6.4.1 via automation Jan 3, 2020
@joeljfischer joeljfischer moved this from In progress to Reviewer approved in v6.4.1 Jan 3, 2020
@joeljfischer joeljfischer removed this from Reviewer approved in v6.4.1 Jan 3, 2020
@joeljfischer joeljfischer added this to Reviewer approved in v6.5.0 Jan 3, 2020
@joeljfischer joeljfischer merged commit b1e31c2 into develop Jan 3, 2020
v6.5.0 automation moved this from Reviewer approved to Done Jan 3, 2020
@joeljfischer joeljfischer deleted the bugfix/issue_1486_show_templateConfiguration_param_not_set branch January 3, 2020 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A defect in the library
Projects
No open projects
v6.5.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants