fixed some rubocop violations

Fixed 2 new eslint errors

Colour preview moved into new browse_tags_helper

In 74d2c4336bab2efe44c6bdadf688d26b18e320b3 browse_helper was split int two

Merge branch 'master' into tag-colour-preview-rebase

Localisation updates from https://translatewiki.net.

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

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

Make the logo on the error page clickable

Also use the same picture/srcset code as on the main header

Merge pull request #2266 from systemed/patch-2

Add user_<uid> class to diary entry div

Exclude all vendored code from rubocop checking

Drop ancient imported daemon generator

Update bundle

Update yarn bundle

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

Update to iD v2.15.5

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Fix numericality validation to use correct integer only constraint

Fixes #2355

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Increase poltergeist timeout

Update rubocop todo

Fix rubocop warnings

Update bundle

Explicitly require gpx in the trace model tests

If we don't do this and no other test manages to trigger an
autoload before we run then we might try and autoload it inside
a MockFS block which will fail.

Revert "Always look to the global scope for the GPX module"

This reverts commit fbefe3550cec6d609ef1dc97ea4c48f73447cee4.

Include any current search terms in the page title

Fixes #2336
Closes #2347

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Create config/settings.local.yml in Vagrant provisioning script

Make sure phantomjs is installed

Fix vagrant provisioning script

Fixes #2343

Exclude phantomjs from generic webkit as it's ancient

Fixes #2339

Patch broken polyfill feature test

https://github.com/Financial-Times/polyfill-service/issues/2006

Update polyfills

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Avoid trying to resize non-resizable images

Fixed #2329

Localisation updates from https://translatewiki.net.

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

Remove file unrelated to iD 2.15.4 release

Update to iD v2.15.4

Pass viewbox to Nominatim when geocoding routing endpoints

Fixes #2327
Closes #2328

Localisation updates from https://translatewiki.net.

Fix text direction for Japanese

Localisation updates from https://translatewiki.net.

Fix new rubocop warnings

Update bundle

Closes #2325

Localisation updates from https://translatewiki.net.

Drop paperclip

Drop old user image columns

Replace migrations that rely on models with rake tasks

Replace some migrations that rely on relationships between
models to add subscriptions with rake tasks.

Don't try and add fingerprints for user images

It won't actually work, and anybody migrating up to the current
schema is never going to use the fingerprints anyway.

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

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

Rework help page text

This avoids using the OSM acronym, and uses better titles than urls
for the help forum and the wiki. Both titles are based on the title
of the linked sites.

Set server_url to test.host in test environments

Fixes #2319

Avoid more models in old migrations

Although these migrations run fine today, they could break in the future
if changes are made to the current model code.

Avoid using live models in old migrations

Instead, we define models in the migrations themselves. This ensures that
column names etc match the state of the database during the migration,
not the current live version of the model.

Update annotations after 91a37235ac748df7dc241b215790d6cbecd652f3

Use the configured host name for user image URLs

Remove support for old style user images

Update gravatar methods to check for Active Storage images

Return absolute URLs for user images

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

Defer deletion of avatars to a background job

Increase expiry for Active Storage URLs

Allow BMP images to be transformed

https://github.com/rails/rails/issues/35953
https://github.com/rails/rails/pull/36051

Run any Active Storage jobs in the storage queue

Send trace import result notification immediately

If we delay sending failure notifications then they will fail
because the trace will no longer exist.

As trace imports are running in the background anyway there
doesn't seem to be any good reason to defer the emails.

Fixes #2312

Fix rubocop warning

Stop creating a shared global querystring variable

Localisation updates from https://translatewiki.net.

Skip uploading images for users that have a new style avatar

Add a rake task to move user avatars to Active Storage

Update querystring references

Revert accidental schema changes

It's to_sym not to_symbol

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

Enable a few more eslint checks

Enforce consistent indentation in javascript code

Fix eslint warning

Update eslint

Closes #2307

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

make link in documentation clickable

Localisation updates from https://translatewiki.net.

Ignore the class length of test files

There's no sensible way to refactor a controller test into multiple classes,
so lets exclude them from the class length metrics to reveal other over-length
classes.

Allow configuration of storage server URL for security policy

Monkey patch Active Storage to set content type when uploading to S3

Add support for using S3 as the object store

Move user images to Active Storage with paperclip as a fallback

Add support for Active Storage attachments

Localisation updates from https://translatewiki.net.

Update wiki page data

Fix rubocop warnings

Update bundle

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

Localisation updates from https://translatewiki.net.

Ubuntu 18.04 is now required. Add apt-get update

Include bundler in packages to install on Ubuntu