Skip to content

Remove temp PIR pixels#4432

Open
quanganhdo wants to merge 1 commit intomainfrom
anh/shared/pir/remove-typed-fallback-temp-pixel
Open

Remove temp PIR pixels#4432
quanganhdo wants to merge 1 commit intomainfrom
anh/shared/pir/remove-typed-fallback-temp-pixel

Conversation

@quanganhdo
Copy link
Copy Markdown
Member

@quanganhdo quanganhdo commented Apr 10, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1205278999335242/task/1213494024530409?focus=true
Tech Design URL:
CC:

Description

Removed temp pixels.

Testing Steps

  1. CI passes

Impact and Risks

What could go wrong?

Quality Considerations

Notes to Reviewer


Internal references:

Definition of Done | Engineering Expectations | Tech Design Template


Note

Low Risk
Low risk: removes debug/telemetry-only pixel firing and associated metadata without changing core scan/opt-out execution flow. Main risk is reduced observability if the typed-fallback situation still occurs in production.

Overview
Removes the temporary telemetry for detecting when broker action payloads fall back from raw JSON to typed encoding.

This deletes the actionPayloadTypedFallbackUnexpected pixel end-to-end (shared pixel enum/mapping, iOS/macOS pixel definitions) and removes the opt-out email-confirmation continuation exception tracking from ActionsHandler, along with the corresponding unit tests.

Reviewed by Cursor Bugbot for commit e8e8c85. Bugbot is set up for automated code reviews on this repo. Configure here.

@quanganhdo quanganhdo force-pushed the anh/shared/pir/remove-typed-fallback-temp-pixel branch from d99ebea to e8e8c85 Compare April 10, 2026 16:04
@quanganhdo quanganhdo requested a review from edulpn April 10, 2026 16:05
@github-actions
Copy link
Copy Markdown
Contributor

This PR has been inactive for more than 7 days and will be automatically closed 7 days from now.

@github-actions github-actions bot added the stale label Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant