Ensure database is writable before destroying a message

Rename messages#read to #show

Also rename the named path, to align with resourceful routing

Rename messages#delete to destroy

This is a better fit for rails controller method conventions, paving
the way to resourceful routing. Keep user-facing strings as 'delete'
since that's more commonly used.

Remove unused POST method for message replies

This action just shows the pre-filled form, with the target for
the form being the same as for new messages.

Update to iD v2.8.2

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

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

Quote translations to stop them looking like numbers

Localisation updates from https://translatewiki.net.

Make CSP rule for the remote control explicitly allow http

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

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

Quote translations to stop them looking like numbers

Localisation updates from https://translatewiki.net.

Rename message_controller to messages_controller

Lazy lookups for translations in message_controller.rb

Rename trace_controller to traces_controller

More lazy translation lookups for traces

Use named paths for messages

Quote translations to stop them looking like numbers

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Fix translation lookups for roundabout exits

Avoid using "one" and "two" as translation keys

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

Add terminating semicolon to overpass query

Fixes #1843

Localisation updates from https://translatewiki.net.

Separated ordinal roundabout exit text from normal exit text

Merge branch 'convert_roundabout_exits' of https://github.com/jguthrie100/openstreetmap-website into convert_roundabout_exits

Change roundabout exit logic and remove blank string from locale file

Merge branch 'master' into convert_roundabout_exits

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

Update translation keys for renaming of note to notes

Fix syntax error with ruby 2.5

Update xid_to_int4 to use postgres version 1 calling convention

Localisation updates from https://translatewiki.net.

Removed directions from roundabout exit instructions

Added exit counts as seperate keys in locale file

Use rails helpers to construct URLs

Fixes #1817
Closes #1840

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.

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