lsuits / OBSOLETE--DO-NOT-USE--gradebook_moodle

LSU's fork of Moodle for gradebook changes
GNU General Public License v3.0
2 stars 1 forks source link

Add FERPA Warning to Gradebook Export #17

Closed BuddyEthridge closed 11 years ago

BuddyEthridge commented 12 years ago

Need to add the obnoxious, in-your-face warning to the Gradebook exporter.

adamzap commented 12 years ago

Are we sold on the obnoxious, design-breaking warning, or will one of the following alternatives work?

philcali commented 12 years ago

@rrusso, doing a small bit of investigation, grade export pages will use the admin page layout. I wonder if it would be possible to place this warning in the theme with some smarts in the admin page layout, rather than baking it into each export type. This requirement very LSU specific.

What are you thoughts on this?

philcali commented 12 years ago

@adamzap, we must have posted so close together, that I missed your comment:

I'd like option 2? (second one listed), over the first one, because the first one increases the number of clicks to perform a simple grade export.

In this particular case, I'd like to investigate if a local plugin could fill the need. The plugin could build a table that records acceptance times, and contains an rpc script to record such clicks. The theme could then inject the necessary js to launch the modal on admin page layout of grade export type.

Whatever path we choose, I'd like to avoid building this behavior in the Gradebook, if possible.

rrusso commented 12 years ago

I might be able to do it in theme, but it would require a different layout for those pages.

I'll look into this.

Although I LOVE option 2. This way we could have a date/time/ip from which they agreed to the FERPA provisions.

I seriously prefer option 2 over any always there totally ignored ugly red box.

rrusso commented 11 years ago

Fixed a long time ago