search

build3 / LSOA

Learning Sciences Observation App - Vanderbilt University
2 stars 0 forks source link

Add migration to fix rest of the constructs #356

Closed kaniak274 closed 3 years ago

kaniak274 commented 3 years ago

Create new migration, this time for all records which have not been touch by previous one. This time the migration will touch a lot more records. I was trying to find the way to make it more safe but unfortunately I didn't find one. I've looked into the data which Chris's send me, and it seems that a lot of students are marked for only one course. Indeed there are some students which participate or participated in more than one unfortunately.

maticomp commented 3 years ago

Well, I have no remarks with regards to the migration code. On the logic side of things though, as long as you will be in touch with Chris so that he knows the sheer extent of records that will get modifed and he's fine with it, sure! I'd like to make sure you both are on the same page regarding what is the (possible) problem and what is the proposed solution.

kaniak274 commented 3 years ago

Well, I have no remarks with regards to the migration code. On the logic side of things though, as long as you will be in touch with Chris so that he knows the sheer extent of records that will get modifed and he's fine with it, sure! I'd like to make sure you both are on the same page regarding what is the (possible) problem and what is the proposed solution.

Sure I will prepare the explanation, so Chris will know where is the problem and possible danger associated with the solution