Filtering and paging done at DB level

• Status field is no longer Transient, so we can filter on it (requires DB update)
• Filtering and Paging can then be moved to the database, which makes it much more efficient (so no longer loading all requests when the requests controller is called)
• Sorting is done by default on the ID field in DESC
• Tabs on course page were unreadable
• Could not move to next page on Requests list

Jenkins Build SUCCESS

Results available at: Jenkins [Queue #219]

By Jenkins on 2017-09-25T06:49:01 (imported from GitLab project)

By Jenkins on 2017-09-25T06:49:03 (imported from GitLab project)

added 1 commit

• 527eb102 - Could not move to next page in requests list

Compare with previous version

changed the description

Jenkins Build SUCCESS

Results available at: Jenkins [Queue #221]

By Jenkins on 2017-09-25T07:05:24 (imported from GitLab project)

By Jenkins on 2017-09-25T07:05:24 (imported from GitLab project)

added 1 commit

• eb0b3b1d - Status no longer transient, allows for filtering on it, filtering and paging…

Compare with previous version

Jenkins Build SUCCESS

Results available at: Jenkins [Queue #223]

By Jenkins on 2017-09-25T08:14:29 (imported from GitLab project)

By Jenkins on 2017-09-25T08:14:30 (imported from GitLab project)

changed title from Tabs on course page were unreadable to Filtering and paging done at DB level

changed the description

mentioned in issue #44 (closed)

added 1 commit

• f392c649 - Filter on active lab in get next and request list

Compare with previous version

Jenkins Build SUCCESS

Results available at: Jenkins [Queue #225]

By Jenkins on 2017-09-25T13:07:01 (imported from GitLab project)

By Jenkins on 2017-09-25T13:07:01 (imported from GitLab project)

Why do we need this?

By Jesse Donkervliet on 2017-09-25T13:40:13 (imported from GitLab project)

What does this annotation do?

By Jesse Donkervliet on 2017-09-25T13:40:49 (imported from GitLab project)

This verbose syntax is required for caching?

By Jesse Donkervliet on 2017-09-25T13:42:49 (imported from GitLab project)

I only looked at the java changes. I trust the HTML changes are OK.

By Jesse Donkervliet on 2017-09-25T13:43:25 (imported from GitLab project)

ALTER TABLE request ADD status INT(11) NULL DEFAULT NULL AFTER comment; should work