search

voidlily / timeboard

Georgia Tech CoC TA timesheets
4 stars 1 forks source link

Mass Approve goes to seemingly random timesheet #25

Closed jking31cs closed 13 years ago

jking31cs commented 13 years ago

Steps: Login as professor go to signed timesheets click approve for all students click approve and go through popup

You end up at a random timesheet with the approve message at the top. I think it should go back to the signed timesheets index page.

apmonroe commented 13 years ago

the update method of the timesheet controller was kidnapping the approval process because the commit value in the update param was identical for single approval and mass approval. I've changed it to Approve Selected for mass approval and the issue seems to be resolved.