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

More tweaks for RTL email

Fixes #2297

Fix styling of email notifications in RTL languages

Fixes #2297

Add missing space

Revert "Block indexing of diary entries"

This reverts commit fc67e63589536b0d8fa3a92aa7eff57ebeceb3a6.

Fix text direction for traditional chinese

Localisation updates from https://translatewiki.net.

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

Update to iD v2.15.3

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

Fix spellings from aggree to agree

As reported on https://translatewiki.net/wiki/Thread:Support/Found_typo_in_https://translatewiki.net/w/i.php%3Ftitle%3DOsm:Users.terms.tou_explain_html/en

Update list of dependencies to install

Rubocop

Replace respond_to with render xml format

Revert XML default for routes

Fix text direction for traditional chinese

Merge branch 'master' into patch/view_migration

Refactoring

Remove non functioning changeset_cache, user_display_name_cache

Localisation updates from https://translatewiki.net.

Avoid unnecessary arrays

Simplify partial rendering

Avoid using @variables in partials

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

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

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

Split some extremely long strings over multiple lines using squiggly heredocs

I'm not entirely convinced that testing walls of text like this is useful
anyway, but at least this approach makes the lines significantly less long.

Rename @edits variable to @changesets, for clarity

Add support for an unversioned 'versions' API call

This will eventually replace the unversioned 'capabilities' call.
Client applications can use this versions call to find out what versions
are available, if they support multiple versions. The capabilities
of each version, and the format of the capabilities call, is then
able to change between versions. If a client only supports one version
they can jump straight to the versioned capabilities call.

Additionally, supported versions are now a list of version identifiers,
without any implications surrounding min/max values (e.g. guesswork
for supported intermediate versions).

Fixes #2162

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

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

Avoid stating the action to render when it just matches the current action

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

Simplify render calls for changesets

Simplify partial rendering when the partial is named after the model

Rework to ensure views are named after the controller method name

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

Fix text direction for traditional chinese

Update bundle

Localisation updates from https://translatewiki.net.