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

[#12048] Refactor FeedbackResponses and FeedbackResponseComments attributes #12791

Closed
wants to merge 12 commits into from

Conversation

domlimm
Copy link
Contributor

@domlimm domlimm commented Feb 24, 2024

Part of #12048. Fixes #12724

Outline of Solution

FeedbackResponses:

  • Update giver attribute to use User entity instead
  • Update recipient attribute to use User entity instead

FeedbackResponseComments:

  • Update giver attribute to use User entity instead
  • Update recipient attribute to use User entity instead
  • Update lastEditor attribute to use User entity instead

@domlimm domlimm force-pushed the v9-refactor-entities branch 5 times, most recently from 4d2f424 to b213aab Compare February 26, 2024 15:44
@domlimm domlimm added the s.Ongoing The PR is being worked on by the author(s) label Feb 27, 2024
@domlimm domlimm force-pushed the v9-refactor-entities branch from b213aab to 4861d41 Compare March 6, 2024 15:36
@domlimm domlimm force-pushed the v9-refactor-entities branch 3 times, most recently from ab40081 to 424b2ab Compare March 30, 2024 04:22
@domlimm domlimm force-pushed the v9-refactor-entities branch from 29c0c1e to bc5c4f7 Compare April 15, 2024 14:17
@domlimm domlimm force-pushed the v9-refactor-entities branch from 3c27af8 to 868897b Compare April 16, 2024 14:32
@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 74 days). 🐌 😢
Hope someone can get it to move forward again soon...

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

@mingyuanc
Copy link
Contributor

@domlimm Could you fix the failing CI

@nusoss-bot
Copy link

Folks, This PR seems to be stalling (no activities for the past 7 days). 🐌 😢
Hope someone can get it to move forward again soon...

@mingyuanc
Copy link
Contributor

@domlimm reopen it when you are ready to continue with it again

@mingyuanc mingyuanc closed this Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s.Ongoing The PR is being worked on by the author(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor SQL Entities - User, FeedbackResponse, FeedbackResponseComment
3 participants