rwarbelow / the_swot_bot

20 stars 10 forks source link

Attendance screen #263

Closed dugancathal closed 11 years ago

dugancathal commented 11 years ago

Fixes #217.

View the commits included in this request for further details, but in short, this pull request adds an Attendance report that uses the new attendances table. Attendance records are created by the attendance page on /live_class and were refactored to use an after_save hook to create the StudentAction for the attendance.

This pull request is currently live on the staging environment for your viewing pleasure.

If all is acceptable, please let us know as a backup of the database needs to be captured before running the migration in fbe9543. That migration creates a load of new data (~17K rows) and we just want to be cautious before this gets merged in. If you have any questions, contact @iericallen or myself(@dugancathal) directly for further information.

dugancathal commented 11 years ago

Hey Rachel!

I'm glad you decided to merge this and it was a pleasure getting to meet you last night.

I just want to make sure that the database migration went smoothly when you ran db:migrate. For my peace of mind, could you let me know when you have a chance?

Thanks!