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

Refactor SQL Entities - User, FeedbackResponse, FeedbackResponseComment #12724

Open
1 of 7 tasks
domlimm opened this issue Feb 4, 2024 · 0 comments · May be fixed by #12791
Open
1 of 7 tasks

Refactor SQL Entities - User, FeedbackResponse, FeedbackResponseComment #12724

domlimm opened this issue Feb 4, 2024 · 0 comments · May be fixed by #12791
Assignees
@domlimm
Labels
committers only Difficult; better left for committers or more senior developers
Milestone
V9.0.0

Comments

@domlimm
Copy link
Contributor

domlimm commented Feb 4, 2024 •
edited

  • Leverage on Foreign Key / Relations between Entities by a SQL DB for User, FeedbackResponse, FeedbackResponseComment
  • Remove Section attributes if a relation is to exist between Entity T and User
  • Generics for necessary methods
  • Look into possible solutions to be compatible and scalable with existing NoSQL *Data files
  • Update all affected methods / tests based
  • Complete migration for updateStudentCascade
  • Update ERD
@domlimm domlimm added the committers only Difficult; better left for committers or more senior developers label Feb 4, 2024
@domlimm domlimm added this to the V9.0.0-beta.0 milestone Feb 4, 2024
@domlimm domlimm self-assigned this Feb 4, 2024
@domlimm domlimm linked a pull request Feb 24, 2024 that will close this issue
Draft
@ziqing26 ziqing26 modified the milestones: V9.0.0-beta.0, V9.0.0 Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@domlimm domlimm

Labels
committers only Difficult; better left for committers or more senior developers
Projects
None yet
Milestone
V9.0.0
Development

Successfully merging a pull request may close this issue.

[#12048] Refactor FeedbackResponses and FeedbackResponseComments attributes domlimm/teammates
2 participants
@domlimm @ziqing26