Skip to content

Course page#458

Merged
bmcutler merged 6 commits intomasterfrom
course_page
Aug 8, 2016
Merged

Course page#458
bmcutler merged 6 commits intomasterfrom
course_page

Conversation

@ZishanHuang
Copy link
Copy Markdown
Contributor

@ZishanHuang ZishanHuang commented Aug 8, 2016

Course page for instructors, TAs and students:

  • Gradeable title, submission time and grading time info.
  • Click on title to go to instructions url.
  • Click to go to submission page of specific gradeable.
  • Button enabled/disabled according to time.

TODO:

  • Links to editing gradeables page and grading page not added.
  • User group check (instructors, TAs, students) hard-coded.
  • Gradeables not sorted chronologically yet.
  • Shortcuts to open grading for a closed item and releasing grades to be added
  • Date checks might be more efficient.
  • Progress bars to be added.
  • More styling.

Questions:

  • If ta-grading disabled and the item is closed, should TAs still be able to see the item and item listed in being graded?
  • Should TAs see submission open time?

BTW do we still prefer dates on buttons?

@bmcutler
Copy link
Copy Markdown
Member

bmcutler commented Aug 8, 2016

I'll merge this when the tests pass. Brandon will start working through the todo list.

@bmcutler bmcutler merged commit c35a5a1 into master Aug 8, 2016
@bmcutler bmcutler deleted the course_page branch August 8, 2016 20:46
@MasterOdin
Copy link
Copy Markdown
Member

We should make an issue to track these

@MasterOdin MasterOdin mentioned this pull request Aug 11, 2016
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants