-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Mobile SDK multiformat feature #4476
Conversation
✅ Deploy Preview for prebid-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
52e89e4
to
89a8b54
Compare
We need someone from the mobile committee to review this doc. Neither Chris nor I have the background to properly review this. @mmullin, could you suggest someone? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My review is really not authoritative, but I skimmed it. My feedback:
- deprecated things (e.g. DISPLAY) need to be highlighted in the release notes. (I'm assuming DISPLAY still works. Shouldn't remove that until 3.0 IMO)
- I don't think it's intuitive that multiformat adunits are created with a function called
BannerAdUnit
. But I suppose that can be addressed in a future release.
Hi @bretg ! We added info about deprecated items to the release notes on both platforms. And yes, deprecated classes, functions, and enum cases still work as expected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
* Update iOS documentation. * iOS documentation corrections. * Update Android documentation. * Android corrections. * iOS corrections. * Android ad unit formats renaming. * iOS AdFormat case renaming * doc: corrections for iOS * doc: corrections for Android --------- Co-authored-by: Olena Stepaniuk <olena.stepaniuk@postindustria.com> Co-authored-by: Yuriy Velichko <yuriy.velichko@postindustria.com>
* Update iOS documentation. * iOS documentation corrections. * Update Android documentation. * Android corrections. * iOS corrections. * Android ad unit formats renaming. * iOS AdFormat case renaming * doc: corrections for iOS * doc: corrections for Android --------- Co-authored-by: Olena Stepaniuk <olena.stepaniuk@postindustria.com> Co-authored-by: Yuriy Velichko <yuriy.velichko@postindustria.com>
Updated Prebid Mobile documentation for the multiformat feature. Added sultiformat examples.
🏷 Type of documentation