Closed nicolelehrer closed 8 years ago
Looks good @nicolelehrer, left a few questions!
@huebnerob thank you for your review! Ready for another look whenev you can
Looks good @nicolelehrer
@huebnerob thanks again for your review. some changes made before merge include
Ticket
Required for Vimeans only VIM-4074
Ticket Summary
Add our model representation for VOD items (films or series) Augment our representation of video responses to handle videos associated with VOD items (trailer, film, extra content video, or an episode of a series)
Implementation Summary
VIMJSONResponseSerializer
VIMVODItem
VIMVideo+VOD
VIMVideo
to house all VOD related helpersVIMConnection
ondemand
andtrailer
connectionsvideos
connection gives us the the videos for a seriesVIMInteraction
VIMVideoPlayRepresentation
VIMVideoPlayabilityStatus
Some to-do's
expires_time
andpurchase_time
to match the format we already support. For now we use a helper method to parse this one-off format. SeeVIMInteraction
implementation file for details.VIMVideo+VOD
implementation file for details.How to Test
Code review only. See parent PR for more context
If helpful, API notes on VOD https://vimean.atlassian.net/wiki/display/DEV/API