-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Yandex ID System page #5268
Yandex ID System page #5268
Conversation
✅ Deploy Preview for prebid-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I am not sure why, but the module did not appear on |
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.
Sorry, just realized that you also need to add a row into the table at https://docs.prebid.org/dev-docs/modules/userId.html#prebidjs-adapters
91fece7
to
cd645b4
Compare
cd645b4
to
90a224a
Compare
Hi @bretg! I've updated the file according to your recommendation. I've also updated the “new user ID module” checklist according to your comment. Please let me know if anything else needed. Thanks! |
Thanks.
Not sure if this comment is old, but I see Yandex ID on the preview -- https://deploy-preview-5268--prebid-docs-preview.netlify.app/download Note that the linting error here comes from a problem not related to this PR and I don't know how to fix... the linter is confused by the liquid script containing a pipe symbol. For now, ignoring the lint issue. We'll merge these docs when the code is released. |
@bretg yep! The download page worked as expected after I updated the adapters table. Thanks a lot! |
@chernodub can you resolve the merge conflicts? |
@muuki88 thanks! I've made the updates. Please take a look when you can :) |
Awesome 😎 |
* feat: add yandex id system docs * fix: add yandex id to adapters table * fix: update docs on how to add user id module * fix: adjust list type * Add storage config
Hi! This PR accompanies the one submitted in Prebid.js repo: prebid/Prebid.js#11196
We have added the Yandex ID system.
If any details needed, we'd be happy to provide it, thanks!
🏷 Type of documentation
📋 Checklist