search

edfan / firehose

A class-picking website for MIT students.
35 stars 11 forks source link

Renaming a non-class activity duplicates the activity #22

Closed talkon closed 2 years ago

talkon commented 3 years ago

Renaming a non-class activity foo to bar results in two activities foo and bar instead of only bar.

cjquines commented 2 years ago

ok so the relevant code is script.js:813 and then script.js:924. probably morally correct way to fix this would be to remember the previous number of what was added and then when adding a new activity, check if it is set, and then remove the old activity before adding the new one?