Code review comment for lp:~ivle-dev/ivle/exercise-ui

Revision history for this message
William Grant (wgrant) wrote :

- ivle.database has some XHTML in it! Please obliterate it.
- ExerciseDeleteView.remove_exercise() should be a method of Exercise.
- ExerciseDeleteView.populate() confuses me.
- The authorisation mechanisms for Exercise editing need to be factored out. Probably into Exercise.get_permissions() itself.
- Somebody (probably me) also needs to fix the jQuery symlink (maybe make the installer do it?)

More to come. I didn't get through even 10% of it.

review: Needs Fixing

« Back to merge proposal