-
Notifications
You must be signed in to change notification settings - Fork 216
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
lifetimeMS Separate Argument B/C? #1002
Comments
The The old second-argument-in-seconds technique will be supported for a while as a fallback, for backwards compatibility. The behavior if you call Join a second time is just like it was before: it sets the lifetime of the IG to whatever you say it should be, overwriting whatever the previous expiration was. In this way you can keep extending the lifetime of an IG until 30 days after the person most recently visited the site where you're invoking the Join operation. |
Thanks @michaelkleber that makes sense, can I just confirm the behavior of how this works when
|
Updates ignore the lifetimeMs field, so they can't extend (or shorten) the lifetime of an interest group. Not allowing extending the field is a deliberate behavior, to avoid an interest group to be persisted indefinitely from a single site visit. Only a script on a page the user visits can extend the lifetime of an IG. |
Hello fwends, I was commenting on one of our PRs internally which led to a conversation about how to tell the call to joinAdInterestGroup what the lifetime is. I now see that the Github doc lists lifetimeMs as being one of the properties of the object passed to the call, rather than the second argument; I also see this in the full spec. However we found that the old method seems supported based on both this doc and experiments.
So QQs:
The text was updated successfully, but these errors were encountered: