Need to Drop Students who drop via bCourses

[cycomachead]

if a student is removed from the bCourses roster, they should be removed from the seating tool

[Reimirno]

Need to do import strategy feature

When importing, allow user to specify different import strategies.
Student is always matched by canvas_id.

mainly, two points of ambiguity:

• existing student: "righty" is false; imported student "righty" is blank -> should result be false or blank?
• existing student is missing in imported sheet -> should it get deleted, or leave it there?

I must have seen a well-defined set of importing strategies somewhere (like "overwrite", "integrate" etc). I shouldn't come up with this myself, which will probably turn out inferior.

[cycomachead]

existing student: "righty" is false; imported student "righty" is blank -> should result be false or blank?

I think what probably makes sense if that if there are no prefs, we just update the student attributes. I don't know about righty specifically what makes sense.

existing student is missing in imported sheet -> should it get deleted, or leave it there?

This depends. For gradescope, Canvas sync removes people who came over via Canvas -- e.g. we track if they have a valid canvas ID.

Largely this sounds like we might need to improve the import students follow:

• Option: Assign default to all students
• Option: Drop students not in the import -- IMO this needs to be off by default so you can still update only some options.
• Option: Reset seating assignments

These could apply whether canvas or CSV imports. The last two options should probably require the user approving a warning...

There is a bonus here -- which I don't know how useful it is, but we can maybe consider only removing seats if the added constrains result in a guaranteed failed match. (e.g. you add a 'lefty' constraint to a student assigned in a righty seat...)

[Reimirno]

Yes I am working on something like that