search

WICG / attribution-reporting-api

Attribution Reporting API
https://wicg.github.io/attribution-reporting-api/
Other
367 stars 173 forks source link

Add epsilon section to README #1439

Closed giladbarkan-github closed 2 months ago

giladbarkan-github commented 2 months ago

Adding visibility to epsilon search could help consumers become more acquainted with the feature and help elicit feedback about adding such automation to clients.

(https://wicg.github.io/attribution-reporting-api/#source-registration-json-key-event_level_epsilon)

csharrison commented 2 months ago

Hey Gilad can you point to the spec associated with this feature?

giladbarkan-github commented 2 months ago

Hey Gilad can you point to the spec associated with this feature?

Sure! My only concern would be if the link might change in the future, it's hard to keep track of where it's used.

giladbarkan-github commented 2 months ago

Hey Gilad can you point to the spec associated with this feature?

Sure! My only concern would be if the link might change in the future, it's hard to keep track of where it's used.

Done.

csharrison commented 2 months ago

Oh sorry I am fine with adding it to the explainer, but I would have been fine just adding it to the PR description too :)

giladbarkan-github commented 2 months ago

Oh sorry I am fine with adding it to the explainer, but I would have been fine just adding it to the PR description too :)

Lol, done!

csharrison commented 2 months ago

Hm you linked to event_level_epsilon definition but not the search feature. Where is that specified?

giladbarkan-github commented 2 months ago

Hm you linked to event_level_epsilon definition but not the search feature. Where is that specified?

I'm confused. We haven't spec'd an automated search feature. This README update is to call out functionality available with the flexible event JavaScript script.

csharrison commented 2 months ago

🤦 sorry I totally misread your PR. Apologies