-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove the externalintegrationslinks table #1379
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #1379 +/- ##
==========================================
+ Coverage 90.29% 90.35% +0.06%
==========================================
Files 232 232
Lines 29751 29699 -52
Branches 6797 6786 -11
==========================================
- Hits 26864 26835 -29
+ Misses 1843 1823 -20
+ Partials 1044 1041 -3
☔ View full report in Codecov by Sentry. |
8e4990d
to
d739045
Compare
b874bdf
to
cf60c47
Compare
d739045
to
3d690c5
Compare
cf60c47
to
0696d92
Compare
7371a07
to
42586d2
Compare
0696d92
to
1563020
Compare
Now that v10.0.0 is out, I think we are good to merge this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
Description
This PR removes the now unused
externalintegrationslinks
table.Motivation and Context
As of #1377 the
externalintegrationslinks
table is no longer being used. It is kept around in that PR in case we need to roll back those changes. Once those changes are successfully running everywhere, we should merge this PR to remove the unused table.How Has This Been Tested?
Checklist