Fix new rubocop warnings

Update bundle

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

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

Remove Imperial from hosting credits

Fixes #1944

Localisation updates from https://translatewiki.net.

Fix incorrect reference to 12.04 in installation notes

Set the rubocop target version to 2.3

This allows rubocop to run additional cops that would not apply to
older ruby versions

Remove bogus translation

Localisation updates from https://translatewiki.net.

Remove bogus translation

Localisation updates from https://translatewiki.net.

Test that friends are shown correctly on the user profile

Fixes #1992

Set img.button size to 20x20 for directions.png

Update translation keys for renaming of user#view to user#show

Update bundle

Update bowser to 1.9.4

Update leaflet to 1.3.4

Avoid flickering welcome banner

Make the welcome banner default to being hidden and then show
it (by adding a class) if there is no cookie.

Fixes #1951

Limit map to the maximum zoom of the current base layer

Fix maximum zoom levels for various layers and ensure that the
map can't be zoomed beyond the limit of the current base layer.

Fixes #1978

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

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

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

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

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

Rename CSS class matches for changes in action names

Localisation updates from https://translatewiki.net.

Resolve 34 Rubocop Lint/AmbiguousOperator conflicts

Remove Lint/ShadowingOuterLocalVariable - already fixed

Resolve 96 Rubocop Lint/AmbiguousRegexpLiteral conflicts

Just pass the object, rather than the id, to _path methods where possible

Just pass the user object, rather than the display_name, to the user_path helper

Rename diary_entry#list to #index

Rename user#list to user#index

Use full url escaping when required in trace tests

Update changeset browse view for renaming of changeset#list to changeset#index

Update trace RSS for renaming of trace#view to trace#show

Improve tests for trace RSS feeds

Localisation updates from https://translatewiki.net.

Fix rubocop warnings

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

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

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

Remove temporary routes

These were preserved to avoid users hitting errors, if we deployed
while they were filling in the related forms.

Rename changeset#list to changeset#index

Rename diary_entry#view to diary_entry#show

Rename user#view to user#show

Hide note comments made by deleted users

Fixes #1970

Do not display two coordinates on 'where am I' click

Closes #1968

Localisation updates from https://translatewiki.net.

Require people replying to diary notifications to be subscribed

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

Put the resourceful routing at the top of the list, for clarity

Rename i18n keys track->trace

Rename traces#list to traces#index

Rename traces#view to traces#show

Split out updating a trace into an update action

Avoid multiple returns

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

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

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

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

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

Tidy up control flow in trace creation

Merge pull request #1938 from jguthrie100/fix_no_trace_description_error

Fixes "new trace" validation error

Fix whitespace in locale file

Refactor message creation to use a create action

This makes it more conventional, rather than handling posts to the new action. The posting of the form was also reworked to use a hidden field for the displayname, rather than in the url, again for convention.

Update to iD v2.11.1

Alter the Makefile to be robust against bundler warnings

In bundle < 2.0, warnings are printed to stdout, along with the path information
that we need. This takes the final line of the output and assumes it's the path.

Fixes #1930

Suppress the creation of boot logs from the ubuntu image

The ubuntu image by default creates a logfile during each boot, but this
is uninteresting for our workflow. This switch suppresses the log.

Fix FactoryBot deprecation warnings

Update bundle

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

Only include embed related messages in the embed javascript

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

Tidy whitespace

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

Reverse coordinates order only when called with two nondescript numbers

Adjust test for latlon searching

Add parsing coordinates in lon, lat format

Localisation updates from https://translatewiki.net.

Update to iD v2.11.0

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Only fetch client side translations for the current locale

Include num_changes in changeset xml response

Avoid some accidental manouver type matches

Localisation updates from https://translatewiki.net.

Stick with autoprefixer 8.x for now

Fix bundle dependencies

Replace sass-rails with sassc-rails

http://sass.logdown.com/posts/7081811

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Set the locale when adding a comment to an issue

Use dynamic error pages built through the asset pipeline

Fixes #1241

Localisation updates from https://translatewiki.net.

Remove script sources which are no longer needed by iD

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