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

Release notes for Prebid 8 #4638

Merged
merged 12 commits into from
Jun 15, 2023
Merged

Release notes for Prebid 8 #4638

merged 12 commits into from
Jun 15, 2023

Conversation

patmmccann
Copy link
Collaborator

🏷 Type of documentation

  • new bid adapter
  • update bid adapter
  • new feature
  • text edit only (wording, typos)
  • bugfix (code examples)
  • new examples

📋 Checklist

@netlify
Copy link

netlify bot commented Jun 13, 2023

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 612c379
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/648a16d4d28cb000088593ae
😎 Deploy Preview https://deploy-preview-4638--prebid-docs-preview.netlify.app/dev-docs/pb8-notes
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@patmmccann patmmccann marked this pull request as ready for review June 14, 2023 17:46
@patmmccann patmmccann requested a review from dgirardi June 14, 2023 18:01
| Google Analytics Adapter | Recommend publishers explore the Generic Analytics Adapter
| Inmar Bid Adapter
| Inskin Bid Adapter
| MASS Deal Rendering Module | Replaced by documentation on bids back handler
Copy link
Contributor

Choose a reason for hiding this comment

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

not clear from this where I should go look for the documentation.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

not merged yet actually #4535

dev-docs/pb8-notes.html Outdated Show resolved Hide resolved
dev-docs/pb8-notes.html Outdated Show resolved Hide resolved
dev-docs/pb8-notes.html Outdated Show resolved Hide resolved

Following the [OpenRTB 2.6 spec](https://github.com/InteractiveAdvertisingBureau/openrtb2.x/blob/main/2.6.md) and the [imp.ext.tid] (https://github.com/InteractiveAdvertisingBureau/openrtb/blob/master/extensions/community_extensions/per-imp-tids.md) community extensions, several fields in Prebid have equivalents in OpenRTB.

$.id, the top level id, is now unique to each bid request. It may come from a bidder `getUUID` call or from the `bidderRequestId`.
Copy link
Contributor

Choose a reason for hiding this comment

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

  • this is of interest to developers more than publishers (not sure if it means it shouldn't be here).
  • I think most readers will have no context to understand either getUUID or bidderRequestId (I wouldn't be suprised if those are not mentioned anywhere else in the docs). they're also not that important IMO, the crucial point is that they should be unique to a single request.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think developers working at bidders may read these notes

@patmmccann patmmccann requested a review from dgirardi June 14, 2023 18:49
@muuki88 muuki88 merged commit 3d771ed into master Jun 15, 2023
4 checks passed
@muuki88 muuki88 deleted the prebid8 branch June 15, 2023 14:02
ecdrsvc pushed a commit to ecdrsvc/prebid.github.io that referenced this pull request Sep 6, 2023
* Update release-notes.md

* Create pb8-notes.html

* Update pb8-notes.html

* Update pb8-notes.html

* Update pb8-notes.html

typo

* Update pb8-notes.html

* Update pb8-notes.html

* Update pb8-notes.html

* Update pb8-notes.html

* Rename pb8-notes.html to pb8-notes.md

* wordsmithing

* added links

---------

Co-authored-by: Demetrio Girardi <[email protected]>
Co-authored-by: bretg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants