Skip to content

Move IEventService to Dirt#7272

Open
eliykat wants to merge 1 commit intomainfrom
ac/move-event-service
Open

Move IEventService to Dirt#7272
eliykat wants to merge 1 commit intomainfrom
ac/move-event-service

Conversation

@eliykat
Copy link
Member

@eliykat eliykat commented Mar 21, 2026

🎟️ Tracking

Follow up from https://bitwarden.atlassian.net/browse/PM-28484

📔 Objective

Event code was moved to DIRT, but I missed IEventService which is still in Admin Console team code. Moving it now.

📸 Screenshots

@eliykat eliykat requested review from a team as code owners March 21, 2026 04:47
@eliykat eliykat requested a review from BTreston March 21, 2026 04:47
@eliykat eliykat enabled auto-merge (squash) March 21, 2026 04:47
@sonarqubecloud
Copy link

@github-actions
Copy link
Contributor

Logo
Checkmarx One – Scan Summary & Details0a5ce230-639e-46e6-8b9d-38b6e4807c9e


New Issues (1) Checkmarx found the following issues in this Pull Request
# Severity Issue Source File / Package Checkmarx Insight
1 HIGH CVE-2026-32933 Nuget-AutoMapper-12.0.1
detailsRecommended version: 15.1.1
Description: AutoMapper is vulnerable to a Denial-of-Service (DoS) attack. Versions prior to 15.1.1 and 16.x prior to 16.1.1, when mapping deeply nested object ...
Attack Vector: NETWORK
Attack Complexity: LOW
Vulnerable Package

Fixed Issues (1) Great job! The following issues were fixed in this Pull Request
Severity Issue Source File / Package
HIGH Cxb9102459-6a3a Nuget-AutoMapper-12.0.1

@codecov
Copy link

codecov bot commented Mar 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.72%. Comparing base (514d8a1) to head (e3b55fc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7272   +/-   ##
=======================================
  Coverage   57.72%   57.72%           
=======================================
  Files        2042     2042           
  Lines       89817    89817           
  Branches     7985     7985           
=======================================
  Hits        51844    51844           
  Misses      36115    36115           
  Partials     1858     1858           

☔ 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.

1 participant