Remove code to set valueAttr at runtime

It does not need doing anymore since tabs were refactored *a long time ago*. We can just set on initialize.
Consolidate table module config on server

There's no need for any of it to live in /common because it's purely server-side
Converted completion_numbers collection to use more appropriate config

By using standard module config params, instead of trying to re-hack the completion-rate params then we can make the completion_numbers module a lot saner.

In doing so, we also add a defult parse method to all collections to do a lot of common operations.
Removed duplicate functionality from collection parse methods

Things which can be delegated up to the global parse method now are, and only really module specific data stuff is left on the child collections.
Make the setting of collect params a little more configurable

Using valueattr for collect by default makes it more complicated to set it in child classes.
Make parse code more readable

Nested underscore methods were a bit confusing. Breaking them out into vars just makes the code a little easier to follow.
Remove list collection

It doesn't do anything. Everything that inherits from it works just as well inheriting from the top-level collection.
Remove group model

It's no longer needed when we work with flat data by default
Move defaulting of timestamps into base collection

Meaning more funcitonality is standard across collections and less is module-specific.
Sort collections by timestamp by default
Move maxlength-ing of bar charts into view

Concatenating the collection at render time allows the module to use an entirely standard collection
Move defaulting of parameters in journey collections up to module level
Standardise config between bar_chart and completion numbers module

Allows the two visualisations of a single data point over time to be used interchangeably.
WIP - remove unnecessary collection methods