martzcodes / gt-course-surveys

Helping students assess course difficulty and workload.
https://omscentral.com
36 stars 11 forks source link

Unable to update profile specialization #40

Closed Zooce closed 7 years ago

Zooce commented 7 years ago

While trying to update my specialization in my profile to Computational Perception and Robotics, the "Updated" message pops up as if everything worked correctly, but when I go back to my profile after visiting another page or logging out, the specialization is one of the other specializations, not Computational Perception and Robotics.

Also, I noticed that the first time I tried updating the specialization, I would always get some Firebase error that it found an undefined value in the specialization. To get past that Firebase error, I had to change the specialization to something else (like Machine Learning).

The bottom line: I think there's something wrong with the Computational Perception and Robotics value in Firebase, which is preventing it from being selected.

mehmetbajin commented 7 years ago

Great find! Classic case of 0 being interpreted as falsy. Fixed with latest deploy just now.