Markus should remember filter/display options on the grader view #293

Closed
NelleV opened this Issue Mar 25, 2011 · 4 comments

Comments

Projects
None yet
3 participants
@NelleV
Member

NelleV commented Mar 25, 2011

Right now, Markus doesn't remember display options chosen for the grader view:
Selecting display: 100 students per page, and asking markus to retrieve the submission for a group reloads the page, and "forgets" the option.

I thought a ticket for that existed already, but I couldn't find it.

@lejaokri

This comment has been minimized.

Show comment
Hide comment
@lejaokri

lejaokri Jun 5, 2012

Contributor

ill take this one..

Contributor

lejaokri commented Jun 5, 2012

ill take this one..

@lejaokri

This comment has been minimized.

Show comment
Hide comment
@lejaokri

lejaokri Jul 11, 2012

Contributor

posting to the correct issue this time..

Hi Severin,

This is regarding review http://review.markusproject.org/r/1253/ .

Regarding the test, I am using cookies to save the per_page attribute. I
was in the
middle of testing the cookies but unfortunately, the command line that runs
the testcases do not seem to detect cookies.

I also saw the following comments
on test/functional/main_controller_test.rb

#bypass cookie detection in the test because the command line, which is
running the test, cannot accept cookies

*
*
Would it be possible to add to the submission_collector_manual_test_script
instead?

Thanks

Contributor

lejaokri commented Jul 11, 2012

posting to the correct issue this time..

Hi Severin,

This is regarding review http://review.markusproject.org/r/1253/ .

Regarding the test, I am using cookies to save the per_page attribute. I
was in the
middle of testing the cookies but unfortunately, the command line that runs
the testcases do not seem to detect cookies.

I also saw the following comments
on test/functional/main_controller_test.rb

#bypass cookie detection in the test because the command line, which is
running the test, cannot accept cookies

*
*
Would it be possible to add to the submission_collector_manual_test_script
instead?

Thanks

@jeffling jeffling referenced this issue Jul 25, 2012

Merged

Issue 293 #792

@jerboaa

This comment has been minimized.

Show comment
Hide comment
@jerboaa

jerboaa Sep 29, 2012

Member

Closed with PR #792

Member

jerboaa commented Sep 29, 2012

Closed with PR #792

@jerboaa jerboaa closed this Sep 29, 2012

@ghost ghost assigned lejaokri Sep 29, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment