Skip to content

ReportingAPI: PermissionPolicyViolationReport#43783

Merged
hamishwillee merged 4 commits into
mdn:mainfrom
hamishwillee:permissions_policy_violation_report
Apr 20, 2026
Merged

ReportingAPI: PermissionPolicyViolationReport#43783
hamishwillee merged 4 commits into
mdn:mainfrom
hamishwillee:permissions_policy_violation_report

Conversation

@hamishwillee

Copy link
Copy Markdown
Collaborator

This adds docs for Permissions-Policy violation reports (PermissionsPolicyViolationReport), along with changes I thought necessary to make it work.

This contributes to #43782 (and #43688)

@hamishwillee hamishwillee marked this pull request as ready for review April 14, 2026 07:48
@hamishwillee hamishwillee requested review from a team as code owners April 14, 2026 07:48
@hamishwillee hamishwillee requested review from pepelsbey and removed request for a team April 14, 2026 07:48
@github-actions github-actions Bot added Content:WebAPI Web API docs Content:HTTP HTTP docs size/m [PR only] 51-500 LoC changed labels Apr 14, 2026
Comment thread files/en-us/web/api/reportingobserver/reportingobserver/index.md
Comment thread files/en-us/web/http/reference/headers/reporting-endpoints/index.md
@github-actions

github-actions Bot commented Apr 14, 2026

Copy link
Copy Markdown
Contributor

Preview URLs (5 pages)

Flaws (1)

Note! 4 documents with no flaws that don't need to be listed. 🎉

Found an unexpected or unresolvable flaw? Please report it here.

URL: /en-US/docs/Web/API/ReportingObserver/ReportingObserver
Title: ReportingObserver: ReportingObserver() constructor
Flaw count: 1

  • macros:
    • Macro domxref produces link /en-US/docs/Web/API/COOPViolationReport which doesn't resolve

(comment last updated: 2026-04-20 00:54:31)

@pepelsbey pepelsbey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! Thank you 🙂

A few suggestions below.

Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
hamishwillee and others added 3 commits April 17, 2026 10:38
Co-authored-by: Vadim Makeev <hi@pepelsbey.dev>
Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
@hamishwillee hamishwillee force-pushed the permissions_policy_violation_report branch from accb42f to 127984f Compare April 17, 2026 00:38
@hamishwillee

Copy link
Copy Markdown
Collaborator Author

@pepelsbey Thanks very much for the review. Fixed as you suggested and suggestions taken. Ready for another look.

@pepelsbey pepelsbey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

All good! Though I would suggest adding the comment slightly differently.

Comment thread files/en-us/web/api/permissionspolicyviolationreport/index.md Outdated
Co-authored-by: Vadim Makeev <hi@pepelsbey.dev>
@hamishwillee hamishwillee merged commit a019b32 into mdn:main Apr 20, 2026
10 checks passed
@hamishwillee hamishwillee deleted the permissions_policy_violation_report branch April 20, 2026 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTTP HTTP docs Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants