Skip to content

[PM-36561] Send edit event logging #7673

Open
harr1424 wants to merge 1 commit into
mainfrom
tools/pm-PM-36561-Send-edit-event-logs
Open

[PM-36561] Send edit event logging #7673
harr1424 wants to merge 1 commit into
mainfrom
tools/pm-PM-36561-Send-edit-event-logs

Conversation

@harr1424
Copy link
Copy Markdown
Contributor

@harr1424 harr1424 commented May 19, 2026

🎟️ Tracking

https://bitwarden.atlassian.net/browse/PM-36561

📔 Objective

This PR introduces an EventType and control flow in order to log Send edit events.

See companion clients PR: bitwarden/clients#20728

@harr1424 harr1424 requested review from a team as code owners May 19, 2026 15:09
@harr1424 harr1424 changed the title added event type and control flow [PM-36561] Send edit event logging May 19, 2026
@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 59.98%. Comparing base (4a3c204) to head (3ecd18c).

Files with missing lines Patch % Lines
...s/SendFeatures/Commands/NonAnonymousSendCommand.cs 92.30% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7673   +/-   ##
=======================================
  Coverage   59.98%   59.98%           
=======================================
  Files        2133     2133           
  Lines       93731    93744   +13     
  Branches     8311     8313    +2     
=======================================
+ Hits        56226    56237   +11     
- Misses      35527    35528    +1     
- Partials     1978     1979    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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