[#12048] Data migration for feedback entities chain

TEAMMATES / teammates

This is the project website for the TEAMMATES feedback management tool for education

https://teammatesv4.appspot.com/

GNU General Public License v2.0

1.66k stars 3.28k forks source link

[#12048] Data migration for feedback entities chain #13045

Closed marquestye closed 5 months ago

marquestye commented 5 months ago

Part of #12048

Outline of Solution

Added methods for cascading migration of FeedbackSession, FeedbackQuestion, FeedbackResponse and FeedbackResponseComment
Remove @UpdateTimestamp annotation for affected entities

Todo in other PRs:

[ ] SeedDb and verification script for feedback entities
[ ] DeadlineExtension and Instructor migration methods

NicolasCwy commented 5 months ago

@marquestye Did you test that this works? I didn't see any seed methods for FeedbackQuestion, FeedbackResponse and FeedbackResponseComment so wondering if any testing was done

NicolasCwy commented 5 months ago

@marquestye any update on the seed script I mentioned earlier?

marquestye commented 5 months ago

@marquestye Did you test that this works? I didn't see any seed methods for FeedbackQuestion, FeedbackResponse and FeedbackResponseComment so wondering if any testing was done

Nope, this has not been tested. Shall I add it to the todo list in the PR description?

NicolasCwy commented 5 months ago

Alright in the next PR try and seed and do the verification to make sure it works!