Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

Show educators links to Quadbase exercises on Assignment page #345

Open
jpslav opened this issue May 19, 2014 · 4 comments
Open

Show educators links to Quadbase exercises on Assignment page #345

jpslav opened this issue May 19, 2014 · 4 comments
Assignees

Comments

@jpslav
Copy link
Member

jpslav commented May 19, 2014

In OST, all exercises come from Quadbase. OST uses the Quadbase API to pull exercise content and cache it inside OST.

When a student views an assignment, they see a list of resources and a list of exercises to work. When the instructor (or any educator) goes to look at an assignment, all they see are grids of student results, one for each combination of cohort (since different cohorts can get different exercises) and student registration type (registered, auditing, dropped).

Above this "Results" section, there is an "Exercises" section which currently shows educators something like:

This assignment has 14 exercises.

You cannot see or work the exercises because they were not assigned to you.

Let's add a list of exercise links here. We have two options, and for the moment I'd be fine with either. We can just provide raw links back to the Quadbase URLs for the exercises or we can make the links reveal a local OST rendering of the OST cached exercise (using the partials we use elsewhere). If we do the former, let's force the link to open the Quadbase URL in a new tab -- unfortunately the load time of the assignment page can be slow for educators, especially when the results grids get large, so we don't want educators going back and forth to this page. If we do the latter, we should still include the Quadbase links next to the revealed locally-displayed OST cached exercise -- this will let instructors go see other Quadbase information (e.g. solutions) easily.

Note that we'll need one Exercise list for each cohort, since the cohorts can have different exercises in any given assignment.

cc @kjdav @Dantemss

@jpslav
Copy link
Member Author

jpslav commented May 19, 2014

ps - the relevant page is app/views/assignments/show.html.erb. Note that some of this view code is verbose and has way to many "if this then that" sections. We don't want to spend too much time cleaning up this code as the plan is to reimplement this baseline in the near future.

@navilan navilan mentioned this issue May 22, 2014
14 tasks
@kjdav
Copy link
Member

kjdav commented May 22, 2014

JP suggested one approach is to make the links provide a local OST rendering of the OST cached exercise. Is it possible to render all OST-cached exercises on one page? If this is possible, would be great to include question, mc and solutions if any. reason is that if an instructor has an assignment with 25 questions, it's time consuming to open 25 questions in diff tabs.

@jpslav @Dantemss

@navilan
Copy link
Contributor

navilan commented May 22, 2014

@kjdav - I don't see why having one page with all Qs/MCs/Solutions is not doable. @jpslav - I think this will follow the theme of #341 - where we show the aggregated list of responses from the students.

navilan added a commit to navilan/ost that referenced this issue Jun 12, 2014
- Add `family` scope and method to the assignment model.
- Add `_questions` partial to show the exercise questions.
- Add `show_url` param and ability to the `_question` partial.
- Add `authority?` view helper function.
- Use `authority?` view helper function.
- Fix copyrights.

TODO:

- The styles for questions are a little weird.  Leaving thigs
  as is to wait for the UX review.
navilan added a commit to navilan/ost that referenced this issue Jun 12, 2014
- Add `family` scope and method to the assignment model.
- Add `_questions` partial to show the exercise questions.
- Add `show_url` param and ability to the `_question` partial.
- Add `authority?` view helper function.
- Use `authority?` view helper function.
- Fix copyrights.

TODO:

- The styles for questions are a little weird.  Leaving things
  as is to wait for the UX review.
@navilan
Copy link
Contributor

navilan commented Jun 12, 2014

@kjdav / @jpslav - I didn't address the separate exercises page as part of this issue.

navilan added a commit to navilan/ost that referenced this issue Jun 13, 2014
- Remove the newly introduced `family` scope in favor
  of using association accessors.
- Remove extra closing tag (typo).
navilan added a commit to navilan/ost that referenced this issue Jun 28, 2014
- Use local variable in view/controller instead of helper.
- Separate exercise section rendering for instructor and
  student roles.
- Remove memoist gem.
- Remove url partial and inline using `exercise.url`
jpslav added a commit that referenced this issue Jun 29, 2014
#345: Add questions with links to exercises section.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants