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

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

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

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

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

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

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

Lazy translation lookups and key renames for help page.

Lazy translation lookup and key renames for fixthemap page.

Use lazy lookups and rename translation keys for export

Use lazy lookups and rename keys for the copyright page

Use lazy lookups and rename keys for the welcome page.

Use lazy lookups and rename keys for the about page.

Use lazy lookups for site translations

Remove empty and unused view template

Use lazy translation lookups for the oauth controller.

Rename note to notes in i18n lookups

Enables lazy translation lookups for most cases. Additionally, rename
keys for notes.description.* to notes.comment.* to match partial
name.

Use lazy translation lookups for redactions

Requires renaming the translation keys

Use lazy translations where possible

Use lazy lookups for user_blocks translations

Update to iD v2.8.1

Localisation updates from https://translatewiki.net.

Mention error message in OAuth setup for iD

Closes #1830

Localisation updates from https://translatewiki.net.

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

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

Improve formatting

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

Use lazy translation lookups for the user_blocks partials.

Pluralise user_preferences_controller

This is the rails convention for controllers and can make route
generation easier.

http://guides.rubyonrails.org/action_controller_overview.html#controller-naming-convention

Move the default_url_options config to the action_mailer initializer

We can't use Rails.application.config here because the initializers
run after ActionMailer::Base has been set up.

Fix for empty array when querying for local community geometries

Allow changeset source tag to be settable via url api
(see https://github.com/openstreetmap/iD/issues/4899)

Update to iD v2.8.0

Allow trusted addresses for better_errors to be set in the environment

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

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

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

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

Remove error class when right-click-selecting destination

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Alter class to avoid conflict with model test class

Set default_url_options for action_mailer

This saves having to repeat the same host and protocol options
throughout the notifiers

Add better_errors gem for easier debugging

Fix handling of title when replying to diary entries

Refactor @this_user to @user

Historically we used @user for the currently logged in user, but
this was changed to `current_user` in 2017.

Merge branch 'master' into moderation

Localisation updates from https://translatewiki.net.

Ask for confirmation before deleting a trace

Fixes #1812

Localisation updates from https://translatewiki.net.

Reworded roundabout exit text

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

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

Tweak highlighting of geolocation errors

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

Added autoPan for Directions markers

Converted roundabouts to take "1st, 2nd, 3rd" exit etc

Added missing onramp instructions to locale file

Moved inline styling to stylesheet

Remove/replace redundant no_place locale translation

Unlocateable place now highlighted in red and specified in error message

The textbox containing the unlocateable place gets highlighted in red
Also added a new locale value to specify an error for a specific place

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

Localisation updates from https://translatewiki.net.

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

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

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

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

Strip trailing whitespace from multiline html translations

Be more consistent with 'my' vs 'your' for objects that belong to
the user

Use named paths for a few trace links

Move the api routes into the api/0.6 scope

Remove unnecessary route aliases

These simply duplicate the aliases that are set by default.

Use concise route definitions for single method routes

Disable image_optim workers in tests

They aren't needed for anything related to testing, and the warnings
about missing binaries adds noise to the test output.

Remove image_optim config loading override

image_optim_rails loads the config files by default, and has a
built-in mechanism for per-environment configs. Forcing the config
loading in this way is unnecessary.

Silence puma startup messages when running tests

Add more tests

Update system tests for report link changes

Avoid sending a commit value from from the search button

Use options_for_select to set the selected items in the search form

Create a report_link helper, and use text instead of a flag icon

Fix new rubocop warnings

Update rubocop

Revert to dalli 2.7.6

Update bundle

Localisation updates from https://translatewiki.net.

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

Localisation updates from https://translatewiki.net.

Convert issue status to an enum

Use user_path links. Fixes #1785

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Refactor set_default_assigned_role to be more easily read.

Redirect to the reportable after creating a report.

Add a redirect and error message if user ends up trying to report something without the correct parameters

Handle issue_types properly if current_user is both and admin and a moderator

Handle current_user being both an admin and a moderator for building user list

Unwind the set_issues method