Skip to content
This repository has been archived by the owner on Apr 26, 2023. It is now read-only.

Added parsing of skip offset for VAST ad #62

Merged
merged 2 commits into from
Feb 12, 2019

Conversation

balavor
Copy link
Contributor

@balavor balavor commented Feb 8, 2019

Changes

  • Added parsing for skip offset. It can be presented as a percentage or as an exact time.
  • To parse the time in seconds from VAST I've added a method for this specific case

@VerizonAdPlatforms/video-partner-sdk-developers: Please review.

@balavor balavor force-pushed the OMSDK-2236/skip-offset-parsing branch from fe13e4d to 21199de Compare February 9, 2019 17:09
@vvp-sdk-bot
Copy link

vvp-sdk-bot commented Feb 9, 2019

1 Warning
⚠️ This PR does not have any reviewers yet.

Current coverage for PlayerCore.framework is 54.54%

No files affecting coverage found


Current coverage for VerizonVideoPartnerSDK.framework is 36.44%

Files changed - -
VASTModel.swift 0.00% 💀
VASTParser.swift 99.71%

Powered by xcov

Generated by 🚫 Danger

sources/advertisements/VASTParser.swift Outdated Show resolved Hide resolved
@balavor
Copy link
Contributor Author

balavor commented Feb 11, 2019

@AndriiMoskvin @VladyslavAnokhin I've added logic to check date format that we get in VAST.

@balavor balavor force-pushed the OMSDK-2236/skip-offset-parsing branch 2 times, most recently from 371f8b7 to 09c0ec9 Compare February 11, 2019 15:08
Copy link
Contributor

@AndriiMoskvin AndriiMoskvin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check my solution in Slack.

@balavor balavor force-pushed the OMSDK-2236/skip-offset-parsing branch 3 times, most recently from 016984b to 75b75a5 Compare February 11, 2019 17:47
Copy link
Contributor

@VladyslavAnokhin VladyslavAnokhin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 nice enum :)

@balavor balavor merged commit 3bcbebd into master Feb 12, 2019
@balavor balavor deleted the OMSDK-2236/skip-offset-parsing branch February 12, 2019 12:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants