"Jump-To" Filter for Marks Spreadsheets is broken #781

Closed
jerboaa opened this Issue Jun 28, 2012 · 6 comments

Comments

Projects
None yet
4 participants
@jerboaa
Member

jerboaa commented Jun 28, 2012

Steps to reproduce:

  1. Create a Marks Spreadsheet
  2. Add a column and an out-of value
  3. Save
  4. Click on the "Grades" tab (the paginated students list should load)
  5. Observe that whatever is selected in the "Jump-To" drop-down, no records are in the result set, which is wrong.
@jerboaa

This comment has been minimized.

Show comment
Hide comment
@jerboaa

jerboaa Jun 28, 2012

Member

It appears that this is a routing issue:

POST http://0.0.0.0:3000/en/grade_entry_forms/1/g_table_paginate?filter=none&sort_by=last_name

404 Not Found
Member

jerboaa commented Jun 28, 2012

It appears that this is a routing issue:

POST http://0.0.0.0:3000/en/grade_entry_forms/1/g_table_paginate?filter=none&sort_by=last_name

404 Not Found
@mikeing2001

This comment has been minimized.

Show comment
Hide comment
@mikeing2001

mikeing2001 Jun 28, 2012

Contributor

I can work on this issue.

Contributor

mikeing2001 commented Jun 28, 2012

I can work on this issue.

@ghost ghost assigned mikeing2001 Jun 29, 2012

@jeffling

This comment has been minimized.

Show comment
Hide comment
@jeffling

jeffling Jul 22, 2012

Member

I fixed this in my UJS branch. it was a simple matter of setting a 'get' to a 'post' in routes.rb. @jerboaa should I my routes.rb in a separate branch ahead of time? it contains several other routing fixes.

Member

jeffling commented Jul 22, 2012

I fixed this in my UJS branch. it was a simple matter of setting a 'get' to a 'post' in routes.rb. @jerboaa should I my routes.rb in a separate branch ahead of time? it contains several other routing fixes.

@jerboaa

This comment has been minimized.

Show comment
Hide comment
@jerboaa

jerboaa Jul 22, 2012

Member

Please separate this fix out and submit it as a pull request on it's own. Fixing routes isn't particularly related to UJS, IMO, anyway. Thanks!

Member

jerboaa commented Jul 22, 2012

Please separate this fix out and submit it as a pull request on it's own. Fixing routes isn't particularly related to UJS, IMO, anyway. Thanks!

@jerboaa

This comment has been minimized.

Show comment
Hide comment
@jerboaa

jerboaa Jul 22, 2012

Member

@mikeing2001 please coordinate with @ummu since you are assigned.

Member

jerboaa commented Jul 22, 2012

@mikeing2001 please coordinate with @ummu since you are assigned.

@benjaminvialle

This comment has been minimized.

Show comment
Hide comment
@benjaminvialle

benjaminvialle Jul 23, 2012

Member

Closed with #794 : #794

Member

benjaminvialle commented Jul 23, 2012

Closed with #794 : #794

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