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

XCH-1994 Add Video Validation #31

Merged
merged 1 commit into from
Oct 13, 2020

Conversation

curlyblueeagle
Copy link
Collaborator

@curlyblueeagle curlyblueeagle commented Oct 7, 2020

There are two changes in this PR

  • Group function that perform actions [validate, build request, interpret response, user sync] together. Makes for readable code
  • Updated validation logic. For rules see Video Implementation Doc . NOTE: This PR only implements validation and not setting default values as described in the document.

@curlyblueeagle curlyblueeagle assigned fchristle, clazy and terryc33x and unassigned clazy Oct 7, 2020
@curlyblueeagle curlyblueeagle merged commit 745c483 into feature/video Oct 13, 2020
carlosfelix pushed a commit that referenced this pull request Sep 10, 2021
…ata (prebid#7167)

* DL-1746: Remove trId and use notid instead (#31)

* Remove trId and use notid instead
* Update version to 0.7.3
* Set default value to empty string

Co-authored-by: SublimeJeremy <jeremy@sublimeskinz.com>

* DL-1780: Fix pubtimeout (#32)

* Update timeout value with onTimeout data
* Add test for onTimeout
* Update state with timeout asap

Co-authored-by: SublimeJeremy <jeremy@sublimeskinz.com>
carlosfelix pushed a commit that referenced this pull request May 7, 2024
* KRKPD-1083: Refactor and expand testing (#31)

* Refactor the Kargo adapter testing to be more comprehensive and include testing of invalid inputs

* Fix the window size test to pull data from the browser instead of hardcoded values

* Remove a duplicate test

* unhook triggerPixel after tests

* fixes circleci failure

* Update the localStorage non-functional test

In FireFox, having localStorage.getItem throw an error isn't working so the test was updated to not have a valid localStorage
value to pass on FireFox.

* Restore the bidderSettings following testing

---------

Co-authored-by: William Klausmeyer <klawil@users.noreply.github.com>
Co-authored-by: “Nick <“nick.llerandi”@kargo.com>
Co-authored-by: William Klausmeyer <willyk95@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants