Skip to content
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

Modify foreign key constraint triggers to cascade #127

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

jujaga
Copy link
Member

@jujaga jujaga commented Feb 29, 2024

Description

This QoL change ensures that referential integrity in the database is preserved in the event records are removed from the database.

SHOWCASE-2625

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the OpenAPI 3.0 v*.api-spec.yaml documentation (if appropriate)
  • I have added necessary documentation (if appropriate)

Further comments

Signed-off-by: Jeremy Ho <jujaga@gmail.com>
@jujaga jujaga added the enhancement New feature or request label Feb 29, 2024
@jujaga jujaga self-assigned this Feb 29, 2024
Copy link

Coverage Report

Totals Coverage
Statements: 96.5% ( 773 / 801 )
Methods: 98.08% ( 102 / 104 )
Lines: 97.96% ( 432 / 441 )
Branches: 93.36% ( 239 / 256 )

@kyle1morel kyle1morel merged commit b80a730 into master Mar 11, 2024
15 of 16 checks passed
@jujaga jujaga deleted the feature/cascade branch March 11, 2024 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants