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

feat(LEO-528): Allow multiple IAB categories in video metadata #5

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

sebmil-daily
Copy link
Collaborator

The same way as we can have an array of IAB categories level 1 in the ORTB request, this PR introduces an array for the IAB categories level 2.

To be forward compatible with level 2.2 and 3.0 specifications, the category IDs should be sent as strings.

The same way as we can have an array of IAB categories level 1 in the ORTB request, this PR introduces an array for the IAB categories level 2.

To be forward compatible with level [2.2](https://github.com/InteractiveAdvertisingBureau/Taxonomies/blob/main/Content%20Taxonomies/Content%20Taxonomy%202.2.tsv) and [3.0](https://github.com/InteractiveAdvertisingBureau/Taxonomies/blob/main/Content%20Taxonomies/Content%20Taxonomy%203.0.tsv) specifications, the category IDs should be sent as strings.
@kvnsw kvnsw merged commit 279f5be into master Mar 25, 2024
kvnsw pushed a commit that referenced this pull request May 7, 2024
* feat: added exadsadserver.md for PR purposes

* fix: after code review

* fix: changed main parameters

* fix: from code review

* fix: clear names

* fix: related to markdown lint validation (#3)

* EX-21135 - from CR (#5)

* fix: related to markdown lint validation

* fix: from code review

* fix: from code review

* EX-21135 - from CR - making URL as relative (#6)

* fix: related to markdown lint validation

* fix: from code review

* fix: from code review

* fix: from CR

---------

Co-authored-by: Pierre Brouca <broucapierre@gmail.com>
sebmil-daily pushed a commit that referenced this pull request May 13, 2024
* feat: added exadsadserver.md for PR purposes

* fix: after code review

* fix: changed main parameters

* fix: from code review

* fix: clear names

* fix: related to markdown lint validation (#3)

* EX-21135 - from CR (#5)

* fix: related to markdown lint validation

* fix: from code review

* fix: from code review

* EX-21135 - from CR - making URL as relative (#6)

* fix: related to markdown lint validation

* fix: from code review

* fix: from code review

* fix: from CR

* fix: changed bidder code

---------

Co-authored-by: Pierre Brouca <broucapierre@gmail.com>
Co-authored-by: tfoliveira <tfoliveira@users.noreply.github.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.

2 participants