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

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

Localisation updates from https://translatewiki.net.

Allow indexing of diary entries again

Allow indexing of diary entries again

Fix importing of GPX traces with a leading byte order marker

Fixes #2258

Localisation updates from https://translatewiki.net.

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

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

Fix new eslint warnings

Replace eslint-rails-ee with direct calls to eslint

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

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

Use built-in form submit translation keys for traces

Use built-in form submit translation keys for oauth client applications

Use i18n for submit button for issue comments

Normally when an en-GB.yml translation is missing, rails falls back to en.yml. But when
using the submit helpers, if the en-GB translation is missing, rails knows how to create
a fallback like 'Create {model_name}' without touching the en.yml file. This string might
then be different from what the test expects, e.g. 'Add Comment'.

So it's important to set the language headers, to avoid phantomjs from picking up your desktop
preferences in this specific case.

Use built-in form sumbit translation keys for diary comments

Also use a diary comment object as the basis for the form

Use built-in form sumbit translation keys for messages

Use built-in form sumbit translation keys for redactions

Use built-in form sumbit translation keys for user blocks

Add a note about avoiding commits to other locale files

Round up the frame size for trace animations

Include first frame twice in trace animations

Test import of various trace formats

Fixes #2254

Fix rubocop warning

Install libarchive-dev for travis

Parse each GPX file in an archive separately

Fixes #2253

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

Update to released version 0.4.0 of gd2-ffij

Localisation updates from https://translatewiki.net.

Fix erblint warnings

Make hidden diary entries and comments more visible

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

Localisation updates from https://translatewiki.net.

Show all diary comments to administrators

Allow admins to unhide diary comments, if they wish

Show deleted diary entries to administrators, if the user isn't also deleted

This will allow administrators to review diary entry deletions from non-spam users.

Allow administrators to unhide diary entries, if they wish

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

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

Ignore the block length of the routes

Since the routes are more of a dsl than code, we've been happy to
repeatedly bump up this number. But then that hides other code which
has over-long block lengths hiding behind.

So instead let's exclude the routes from this metric.

Remove dead link to sitemap

Fixes #1113

Fix rubocop warning

Update rubocop todo

Add rubocop-rails

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

Update bundle

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

Use strong_migrations to help developers avoid problems in production database schema changes

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

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

rubocop and erblint fixes

Allow a configurable delay to keep diary entries from being published in feeds

This is to provide another tool to help spam fighting.

The configuration value is set to zero hours for new deploys in
order to be 'least surprising'. You can change the setting without
interfering with your test suite.

Refactor diary entry form

This uses the built-in helpers for naming form elements, rather than
having to override with specific i18n labels. It also changes the label
to 'Update', and capitalizes the title on the edit page.

Refactor some diary entry routes to use resources

Refactor diary entries to use create and update methods

This brings it slightly more into line with resourceful routing.

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

Update annotations to show 'using' clause on indexes

Update annotations to correctly show bigint types

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