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.

Sign up for GitHub

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

Jump to bottom

Adds missing foreign key and fixes constraint reference when delete #616

Merged
merged 2 commits into from
Oct 5, 2021
Merged

Adds missing foreign key and fixes constraint reference when delete #616

merged 2 commits into from
Oct 5, 2021

Conversation

subigre
Copy link
Contributor

@subigre subigre commented Oct 1, 2021

Changes

  • Adds the missing foreign key constraints betwenn ehr and folder tables.
  • Removes folder reference before deleting folder associated to an EHR

Related issue

Resolves ehrbase/project_management#318

Additional information and checks

  • Pull request linked in changelog

@sonarcloud
Copy link

sonarcloud bot commented Oct 1, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@subigre subigre marked this pull request as ready for review October 1, 2021 11:23
@chevalleyc chevalleyc merged commit 215e894 into develop Oct 5, 2021
@chevalleyc chevalleyc deleted the bugfix/318_missing_ehr_folder_fk branch October 5, 2021 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chevalleyc chevalleyc chevalleyc approved these changes

Assignees

@chevalleyc chevalleyc

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@subigre @chevalleyc