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.

Sign up for GitHub

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

Jump to bottom

OTT-806 :: Adpod Support (CTV) #532

Merged
merged 130 commits into from
Jul 2, 2024
Merged

OTT-806 :: Adpod Support (CTV) #532

merged 130 commits into from
Jul 2, 2024

Conversation

pm-nikhil-vaidya
Copy link
Collaborator

Description

Please add change description or link to ticket, docs, etc.

Checklist:

  • PR commit list is unique (rebase/pull with the origin branch to keep master clean).
  • JIRA number is added in the PR title and the commit message.
  • Updated the header-bidding repo with appropiate commit id.
  • Documented the new changes.

For Prebid upgrade, refer: https://inside.pubmatic.com:8443/confluence/display/Products/Prebid-server+upgrade

@pm-nikhil-vaidya pm-nikhil-vaidya marked this pull request as draft August 4, 2023 13:03
pm-nikhil-vaidya and others added 20 commits August 12, 2023 19:57
@pm-nikhil-vaidya
support to parse GET calls
e8b42c6
@pm-nikhil-vaidya
Merge branch 'ci' into OTT-806-Adpod-2
38108a4
@pm-nikhil-vaidya
Added Response handling for each endpoint
b7c47ac
@pm-nikhil-vaidya
Added json response handling
e3f3607
@pm-nikhil-vaidya
support for redirect url in json GET endpoint
e310f5d
@pm-nikhil-vaidya
Modified middleware implementation
e83baca
@pm-nikhil-vaidya
Modified the app and device implementation
fa50487
@pm-nikhil-vaidya
Handle writting headers in final response
4989eb9
@pm-nikhil-vaidya
Fixed panic due to devicetype
bde0505
@pm-nikhil-vaidya
fixed seat non bid appending in response
be04815
@pm-nikhil-vaidya
fixed panics
5d78ece
@pm-nikhil-vaidya
Merge branch 'ci' into OTT-806-Adpod-2
4cbbde9 
Merge branch 'ci' into OTT-806-Adpod-2
@pm-nikhil-vaidya
added recovery when middleware panics
d7cc10b
@pm-nikhil-vaidya
Merge branch 'ci' into OTT-806-Adpod-2
7d1b27a 
Merge branch 'ci' into OTT-806-Adpod-2
@pm-nikhil-vaidya
added enabled flag in Ow module config
e79521d
@pm-nikhil-vaidya
Minor changes
c6cc36b
@pm-nikhil-vaidya
Added error logger and updated pointers in middleware
265520f
@pm-nikhil-vaidya
Fixed redirect issue when debug is enabled
8474eaf
@pm-nikhil-vaidya
Added debug targetting in the response
43f99b5
@pm-nikhil-vaidya
Moved targetting into debug mode
25d26de
@pm-nikhil-vaidya pm-nikhil-vaidya marked this pull request as ready for review October 13, 2023 06:50
@pm-nikhil-vaidya pm-nikhil-vaidya changed the base branch from ci_backup_20240402 to ci May 21, 2024 06:10
pm-viral-vala
pm-viral-vala previously approved these changes Jul 1, 2024
View reviewed changes
@ashishshinde-pubm ashishshinde-pubm self-requested a review July 2, 2024 05:14
@pm-viral-vala pm-viral-vala merged commit e39df49 into ci Jul 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pm-jaydeep-mohite pm-jaydeep-mohite pm-jaydeep-mohite left review comments

@ashishshinde-pubm ashishshinde-pubm ashishshinde-pubm approved these changes

@pm-viral-vala pm-viral-vala pm-viral-vala approved these changes

@Pubmatic-Dhruv-Sonone Pubmatic-Dhruv-Sonone Pubmatic-Dhruv-Sonone left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@pm-nikhil-vaidya @pm-jaydeep-mohite @pm-viral-vala @Pubmatic-Dhruv-Sonone @ashishshinde-pubm @Ankit-Pinge @Pubmatic-Supriya-Patil