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

ScreenManager Tries to Upload Soft Button Images on Sync 2.0 #2093

Closed
joeljfischer opened this issue Jun 8, 2022 · 0 comments · Fixed by #2094
Closed

ScreenManager Tries to Upload Soft Button Images on Sync 2.0 #2093

joeljfischer opened this issue Jun 8, 2022 · 0 comments · Fixed by #2094
Labels
bug A defect in the library manager-screen Relating to the manager layer - screen managers

Comments

@joeljfischer
Copy link
Contributor

Bug Summary

It seems the ScreenManager tries to upload images on Sync 2 even if the dynamic images are not supported. That doesn't crash the app but it just sends RPCs when there is no need for that.

See the equivalent Java Suite issue for more details: smartdevicelink/sdl_java_suite#1738

Reproduction Steps

  1. Run SDL app on Sync 2 TDK
  2. Use the SoftButtonManager to set buttons with images

Expected Behavior:
Images for the buttons shouldn't be uploaded because the TDK doesn't support dynamic images.

Observed Behavior:
SoftButtonManager tries to upload the images then fail

iOS Version(s)

No response

sdl_ios Version

7.4.0

Testing Environment(s)

* Ford Sync 2.0 TDK

Relevant log output

No response

@joeljfischer joeljfischer added bug A defect in the library needs triage An issue that has not yet been triaged manager-screen Relating to the manager layer - screen managers and removed needs triage An issue that has not yet been triaged labels Jun 8, 2022
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 manager-screen Relating to the manager layer - screen managers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant