Skip to content

feat: Add telemetry opt-in/out buttons to consent page#145

Merged
spalmurray merged 2 commits intomainfrom
spalmurray/more-button
Jul 15, 2025
Merged

feat: Add telemetry opt-in/out buttons to consent page#145
spalmurray merged 2 commits intomainfrom
spalmurray/more-button

Conversation

@spalmurray
Copy link
Contributor

Another Mozilla requirement. Hopefully this is satisfactory.

@spalmurray spalmurray marked this pull request as ready for review July 15, 2025 14:49
@codecov
Copy link

codecov bot commented Jul 15, 2025

Bundle Report

Changes will increase total bundle size by 7.12kB (1.11%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
codecov-browser-extension-array-push 647.47kB 7.12kB (1.11%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: codecov-browser-extension-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
vendor.js 3.4kB 383.25kB 0.89%
background.js 127 bytes 198.09kB 0.06%
githubFile.js 1.51kB 11.5kB 15.13% ⚠️
githubPR.js 1.35kB 6.67kB 25.48% ⚠️
../consent.html 340 bytes 5.12kB 7.11% ⚠️
../consent.js 389 bytes 1.1kB 54.71% ⚠️

Files in vendor.js:

  • ./src/types.ts → Total Size: 931 bytes

  • ./src/content/common/sentry.ts → Total Size: 1.1kB

Files in githubFile.js:

  • ./src/content/github/common/fetchers.ts → Total Size: 2.44kB

  • ./src/content/github/file/main.tsx → Total Size: 11.75kB

Files in githubPR.js:

  • ./src/content/github/common/fetchers.ts → Total Size: 2.44kB

  • ./src/content/github/pr/main.tsx → Total Size: 6.25kB

@spalmurray spalmurray merged commit 4501fc5 into main Jul 15, 2025
3 checks passed
@spalmurray spalmurray deleted the spalmurray/more-button branch July 15, 2025 15:16
@sentry
Copy link

sentry bot commented Jul 15, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎

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.

2 participants