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

[SDL 0124] SDLImageUploadManager #865

Closed
theresalech opened this issue Feb 9, 2018 · 1 comment

Comments

@theresalech
Copy link
Contributor

commented Feb 9, 2018

Proposal: SDLImageUploadManager

Currently the logic SDL applications must implement to manage image uploads is tedious and error prone. The typical implementation is rewriting images using the same filename any time the image needs to be shown, or uploads all images at launch and never verifies their existence before calling a Show request. This proposal aims to centralize image management logic, optimize uploads with a local cache when possible, and generally move all image management responsibilities into the proxy.

Review: smartdevicelink/sdl_evolution#366

Steering Committee Decision:

The Steering Committee voted to accept this proposal with revisions. The revisions are outlined in the last comment from joeljfischer. There was also discussion about the relationship between this proposal and SDL 0133 - Enhanced iOS Proxy Interface. It was ultimately determined that this proposal could be accepted with revisions as it is relatively low level in comparison with SDL 0133, but the scope of this proposal would also be considered during the Enhanced iOS Proxy Interface workshop.

The proposal .md was updated on February 9, 2018 to include the agreed upon revisions.

@theresalech theresalech added the proposal label Feb 9, 2018

@joeljfischer joeljfischer added this to the 5.2.0 milestone Feb 9, 2018

@joeljfischer joeljfischer added this to In Progress in v5.2.0 Feb 12, 2018

@NicoleYarroch NicoleYarroch referenced this issue Feb 21, 2018
4 of 4 tasks complete

@joeljfischer joeljfischer moved this from In Progress to In Review in v5.2.0 Feb 23, 2018

@theresalech

This comment has been minimized.

Copy link
Contributor Author

commented Mar 6, 2018

Closed via #874

@theresalech theresalech closed this Mar 6, 2018

v5.2.0 automation moved this from In Review to Done Mar 6, 2018

@joeljfischer joeljfischer referenced this issue Mar 16, 2018
2 of 2 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
2 participants
You can’t perform that action at this time.