Skip to content

Conversation

@ceorourke
Copy link
Member

@ceorourke ceorourke commented Dec 2, 2024

Follow up to #81386 to remove usages of the IncidentSeen and IncidentSubscription models before dropping them.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 2, 2024
@codecov
Copy link

codecov bot commented Dec 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #81522      +/-   ##
==========================================
+ Coverage   80.41%   80.43%   +0.01%     
==========================================
  Files        7241     7241              
  Lines      321573   321539      -34     
  Branches    20830    20830              
==========================================
+ Hits       258604   258631      +27     
+ Misses      62574    62513      -61     
  Partials      395      395              

@ceorourke ceorourke force-pushed the ceorourke/rm-incidentseen-incidentsubscription branch from 844bd98 to c5273e4 Compare December 2, 2024 21:56
@ceorourke ceorourke marked this pull request as ready for review December 2, 2024 21:57
@ceorourke ceorourke requested a review from a team as a code owner December 2, 2024 21:57
@ceorourke ceorourke merged commit 0810372 into master Dec 3, 2024
51 checks passed
@ceorourke ceorourke deleted the ceorourke/rm-incidentseen-incidentsubscription branch December 3, 2024 18:18
ceorourke added a commit that referenced this pull request Dec 3, 2024
…81605)

Another follow up to #81522 to
remove the unused models. A final PR will follow to drop the tables.
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants