search

srobo / nemesis

User management web interface
https://studentrobotics.org/userman/
1 stars 2 forks source link

2208: College refresh should also pick up added/removed members #4

Closed Krenair closed 10 years ago

PeterJCLaw commented 10 years ago

This doesn't contain any tests, which I think it probably could (especially as the front-end test framework is now functional).

It also requires knowledge of the id of the edit field, which it shouldn't (and actually doesn't need to), and it doesn't clear the url hash, which it would be suitable to do here too.

PeterJCLaw commented 10 years ago

I've put together a fix based on this which will include the above elements. Thanks for the partial solution.