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

[SQL] Rename project_rel table and add constraints #4492

Merged
merged 6 commits into from
Jul 22, 2019

Conversation

ridz1208
Copy link
Collaborator

@ridz1208 ridz1208 commented Apr 30, 2019

Brief summary of changes

This renames the project_rel table to project_subproject_rel to respect the SQLmodeling standards

This also adds a front-end control to associate subprojects to projects from the project configuration page. (this should simplify testing)

other changes in this PR

  • add foreign key constraints to the Project and subproject tables for integrity
  • change fields to match their respective foreign keys

@ridz1208 ridz1208 added Cleanup PR or issue introducing/requiring at least one clean-up operation Language: SQL PR or issue that update SQL code labels Apr 30, 2019
@@ -85,7 +85,7 @@ class ProjectDefault extends Project
}

/**
* Get that project subprojects as defined in the project_rel table
* Get that project subprojects as defined in the project_subproject_rel table
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not accurate. Project default gets all subprojects because it is meant to be used when there is no project

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise, it seems fine

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@ridz1208 ridz1208 requested a review from xlecours April 30, 2019 22:32
@ridz1208 ridz1208 force-pushed the 2019_04_30_rename_project_rel branch 2 times, most recently from 52af2ff to 3a59ab4 Compare May 2, 2019 21:58
@ridz1208 ridz1208 added the State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed label May 2, 2019
@ridz1208
Copy link
Collaborator Author

ridz1208 commented May 2, 2019

blocked by #4493

@ridz1208 ridz1208 changed the base branch from major to 21.0-release May 23, 2019 16:50
@ridz1208 ridz1208 force-pushed the 2019_04_30_rename_project_rel branch from 62b0fc9 to 414355d Compare May 23, 2019 16:59
@ridz1208
Copy link
Collaborator Author

DO NOT MERGE. its on the wrong base for now

@ridz1208 ridz1208 added the State: Needs work PR awaiting additional work by the author to proceed label May 29, 2019
@ridz1208 ridz1208 force-pushed the 2019_04_30_rename_project_rel branch from 79b1066 to b04f545 Compare July 3, 2019 18:02
@ridz1208 ridz1208 removed the State: Needs work PR awaiting additional work by the author to proceed label Jul 3, 2019
@ridz1208 ridz1208 force-pushed the 2019_04_30_rename_project_rel branch from 4a4f218 to 1095fac Compare July 3, 2019 23:42
@ridz1208 ridz1208 changed the base branch from 21.0-release to minor July 11, 2019 15:24
@ridz1208 ridz1208 removed State: Blocked PR or issue awaiting an external event such as the merge or another PR to proceed labels Jul 11, 2019
@johnsaigle
Copy link
Contributor

Can you add testing instructions?

@johnsaigle johnsaigle added the Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...) label Jul 16, 2019
PapillonMcGill
PapillonMcGill previously approved these changes Jul 17, 2019
Copy link
Contributor

@PapillonMcGill PapillonMcGill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pending Travis.

@driusan driusan merged commit 7a1d45e into aces:minor Jul 22, 2019
@ridz1208 ridz1208 added this to the 21.1.0 milestone Jul 22, 2019
zaliqarosli pushed a commit to zaliqarosli/Loris that referenced this pull request Aug 28, 2019
This renames the project_rel table to project_subproject_rel to respect the SQLmodeling standards.
@ridz1208 ridz1208 modified the milestones: 21.1.0, 22.0.0 Nov 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Documentation PR or issue that aims to improve the documentation (test plans, wiki, comments...) Cleanup PR or issue introducing/requiring at least one clean-up operation Language: SQL PR or issue that update SQL code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants