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

Add details about server orchestrated multi seller flow for PAS auctions #184

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jasarora-google
Copy link
Collaborator

No description provided.

encrypted Protected Audience request payload. The top-level seller’s code in the publisher web page asks the browser
for this encrypted Protected Audience data to include in the unified request.
encrypted Protected Audience request payload. The top-level seller’s code in the publisher web page/app asks the browser/android API
for the encrypted Protected Audience/Protected App Signals data to include in the unified request.
Copy link
Member

Choose a reason for hiding this comment

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

Should we instead do the following here:

s@Protected Audience/Protected App Signals@Protected Auction(Protected Audience OR Protected App Signals)@ and then use Protected Audience everywhere else below?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It sounds good to me, but does have the risk of adding a new term in the middle of the explainer. Have we used Protected Auction in other explainers to refer to both as a group?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sounds good. will update today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants