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

Database: FAC Cohort itself could be PRIMARY KEY #125

Open
helenzhou6 opened this issue May 16, 2018 · 0 comments
Open

Database: FAC Cohort itself could be PRIMARY KEY #125

helenzhou6 opened this issue May 16, 2018 · 0 comments

Comments

@helenzhou6
Copy link
Collaborator

helenzhou6 commented May 16, 2018

Lots of SQL queries need to be changed, since they are based on the fac cohort id.

I do think that the FAC Cohort should be it's separate table though, since when at the filtering stage, when you want to add a FAC Cohort to filter by, there should be an autocomplete input box (that uses SQL query SELECT * FROM fac_cohort to get a list of all the available FAC cohorts - which would be more difficult to do if it wasn't in it's separate table)

Related to refactor list of things to do - issue #59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant