Try to make erblint happy

Add checkbox for contributor terms and improve text

Remove unnecessary test for 0

Fix tests for change in users_controller

Rubocop fixes and increase per method complexity limit from 23 to 25

Style fixes

Fix tests for change in users_controller, add test for missing read_tou

Require that tou_read checkbox is set

Use jquery

Rename string to avoid translation confusion

Move link to ToU to attribution footer

Move onclick handling of tou checkbox to ujs

Add links to the ToU and include them in signup

- this adds links to the ToU in the Welcome box and on the About page
- adds a timestamp column to the users table to record acceptance of the ToU
- require ToU to be accepted before signup can proceed
- rearrange and change text on signup page

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update bundle

Update polyfills

Update leaflet to 1.5.1

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Remove version constraint on postgres driver

Add geolink class to login/logout referer links

Fixes #2216

Localisation updates from https://translatewiki.net.

Teach travis to validate the database schema

Update database schema

Use rails 4.2 defaults for older migrations

Avoid error when there is no layer with a keyid

Fixes #2215

Return correct base layer ID

Add rubocop-performance

Fix new rubocop warnings

Disable failing cop pending upstream fix

Update bundle

Revert to tracking new versions of rinku

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Switch travis to use xenial

Merge remote-tracking branch 'upstream/pull/2205'

Localisation updates from https://translatewiki.net.

Resourceful(ish) naming for api/users_controller

Resourceful route names for api/trace_controller

Localisation updates from https://translatewiki.net.

Enable a number of eslint style rules

Enable some more eslint checks

Exclude generated i18n files from eslint checks

Use double quotes consistently in javascript code

Enable some more eslint rules from iD rules

Localisation updates from https://translatewiki.net.

Fix queuing of trace imports for API uploads

Fixes #2200

Merge remote-tracking branch 'upstream/pull/2196'

Fix issues found by eslint

Replace jshint with eslint

Localisation updates from https://translatewiki.net.

Use erb-lint to prohibit inline scripting

Format closed_at correctly in changeset API responses

Fixes #2195

Localisation updates from https://translatewiki.net.

Fix dependencies

Update to rails 5.2.3

Expect CORS preflight responses to not have a content type

https://github.com/cyu/rack-cors/pull/180

Update bundle

Merge remote-tracking branch 'upstream/pull/2190'

erblint: use consistent spacing in HTML tags

erblint: remove leading blank lines

erblint: avoid dynamic find_by methods

erblint: add rel=noopener to links that open in a new tab

erblint: use conditional assignment

erblint: use word arrays

erblint: use parentheses for nested function calls

erblint: use symbols as procs

erblint: avoid negated if conditions

erblint: prefer symbolic logical operators over and/or/not

Merge remote-tracking branch 'upstream/pull/2192'

Merge remote-tracking branch 'upstream/pull/2191'

Disentangle the api abilities from the web abilities

This will allow us to rename api actions without causing permissions headaches. The choice of
abilities files is made by inheriting from either api_controller or application_controller.

Also rename capabilities to api_capabilites, for consistency.

Set image dimensions in order to avoid a reflow after the page is drawn by the browser

Refactor notes#mine to simplify code

erblint: fix remaining single quoted strings

Merge remote-tracking branch 'upstream/pull/2189'

Use trace jobs by default

Merge remote-tracking branch 'upstream/pull/2188'

Use the trace jobs for creating and destroy traces via the API

Merge remote-tracking branch 'upstream/pull/2187'

Check validity token in changesets controller

It makes no difference today, but prevents problems in future if
methods accepting post requests are added to this controller in future.

erblint: avoid parentheses for method calls with no arguments

erblint: prefer double quoted strings

Exclude most views from string literal checks

This allows us to work through them slowly, while preventing errors
from creeping into directories that already pass.

erblint: Add or remove curly braces around hash parameters as required

erblint: use hash rocket syntax

Merge remote-tracking branch 'upstream/pull/2168'

Exclude files in the vendor directory from erb-lint rubocop checks

Fix capitalisation of div tag

Update coding style section of the contribution guide to describe our use of rubocop and erblint

Add erblint to travis

Introduce erb-lint for linting erb files

This also allows running rubocop on the ruby code within the erb
files.

Whitespace fixes to various erb files

Whitespace fixes for messages erb files

Whitespace fixes for layouts erb files

Whitespace fixes for diary_entry erb files

Whitespace fixes for browse erb files