uoft-tapp / old-orig-tapp

TA assignment and matching application.
0 stars 2 forks source link

[hotfix] create enrolment updater #179

Closed michellemtchai closed 7 years ago

michellemtchai commented 7 years ago
simpslandyy commented 7 years ago

Let's re-evaluate if #177 is a hotfix or a feature. @jmzaleski @reidka This requires both front-end and back-end support, and it's an additional feature not a bug. Plus, it looks like this might be deployed later tonight or next week, in which it's bad practice to deploy late Friday afternoon.

simpslandyy commented 7 years ago

Note: @jmzaleski talked to @reidka today, we can contemplate merging, once we've demo'd to Karen

reidka commented 7 years ago

@simpslandyy This is definitely not urgent, especially since CP is much higher priority right now. I thought I might get around to testing this myself.

gabriellesc commented 7 years ago

@reidka I had offered @michellemtchai to review this PR, and I'm happy to do so now - unless you would prefer to?

reidka commented 7 years ago

@gabriellesc you are definitely welcome to review!

gabriellesc commented 7 years ago

:+1: