-
Notifications
You must be signed in to change notification settings - Fork 7
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
Collector script to use Async get eids function #40
Comments
shared-id/pubcid.js/src/index.js Line 9 in 5a5d368
|
Without knowing what gpt expects I would not blindly switch to the async function; it will sometimes provide an id when the sync version wouldn't, but it will be much slower on average. |
So this behavior(signal collection) is built with an asynchronous nature. It is better to populate the signal slowly than populate quickly and missing. |
@patmmccann - what is this issue about? should it be in the shared-id repo? |
Next steps:
|
Should we simply deprecate this script? GAM can now find the sharedid |
prebid/Prebid.js#8201 allows the collection of eids async. This is more likely to resolve.
@jdwieland8282 @SKOCHERI
The text was updated successfully, but these errors were encountered: