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.

Allow signups to be blocked by MX host

Block indexing of diary entries

Update Potlatch 2 to 2.5-83-g28c84372 build

Localisation updates from https://translatewiki.net.

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

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

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

Update note comments to use friendly_date_ago helper

Refs #2255

Use friendly_date_ago for notes display

Refs #2255

Use friendly_date_ago for user blocks

Refs #2255

Use friendly_date_ago helper for changeset comments

Refs #2255

Use friendly_date_ago helper for chnagesets

Refs #2255

Use friendly_date_ago helper for user_blocks

Refs #2255

Removes two unused translations.

Use friendy_date_ago helper for notes

Refs #2255

Add a friendly_date_ago helper

Refs #2255

Use distance_in_words_ago scope for user blocks

Refs #2255

Use scope for common details and changeset display in browse pages

Refs #2255

Use scope for time_ago when viewing users

Refs #2255.

Also removes unused translation referencing ago, that was unneeded
after cbf71f04c244440f82d86e12758a2e443f68c0f8

Use distance_in_words_ago scope for issues display

Refs #2255

Use the distance_in_words_ago scope for diary comments pages

Refs #2255

Use the distance_in_words_ago scope for trace display

Refs #2255

Use user models, not ids, in assertions

Pass the models, not the ids, when dealing with friendships

Rename user.friend_users to user.friends

This is possible now since we've renamed Friend to Friendship

Rename Friend model to Friendship

This better describes what the model represents, which is the relationship
between the two users.

Add translations for distance_in_time_ago

This allows translators to accurately translate times when they
need to modify the distance_in_time beyond just adding 'ago'.

Refs #2255

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

Update bundle

Pass through `photo_overlay` parameter
(re: https://github.com/openstreetmap/iD/issues/6492)

Update to iD v2.15.2