Refactored the code to invoke the message body partial layout, as suggested in the PR

Move attach_project_logo to be its own before_action filter, as suggested

Renamed @user_message_author to @author as suggested

Localisation updates from https://translatewiki.net.

Revert "Added a greeting to the changeset comment message"

This reverts commit 39c3fa427ea273e9011c380f9d2f2a9759cddb24.

Remove that ugly style="margin: 0"

It's really only needed for rendering in Yahoo. Let's try to do without having to copy-paste that into every html template.

Simplified the partial layout

Rather than wrap the body in a :body block, just make it the default, unnamed block, which keeps the markup more lightweight.

Logo was being attached twice on changeset comment notifications

Dressed up note comment messages

Dressed up lost password mail

Dressed up gpx notification mails

Dressed up friend_notification mail

Added a greeting to the changeset comment message

The greeting includes the recipient's user name (as for the message notification mail), which should increase the recipient's trust that the message isn't spam. Greetings were reused from each language's respective locale flie.

Rubocop finds 'return' statements redundant

Modify tests to search only text parts

The tests assumed that every part in the multipart emails that we send were either plain text or HTML. Now we have image attachments, against whose contents the tests were still trying to match regexes. The tests have been modified to only run regexes on the text parts of the mails.

Removed unbalanced tag HTML tag

Don't try and look up traces until the user is logged in

Fixes #1411

Localisation updates from https://translatewiki.net.

Dressed up a few more messages.

email_confirm, message_notification and signup_confirm now use the new templates. This fixes some, but not all, of the currently broken tests.

Moved message table HTMl to a partial template

Several notifications are about a message having been sent from one user to another via OSM. I've moved the code for the HTML table that holds the actual user message, with the avatar and the body text, to its own partial template, so that it can be reused. I've updated a second notifier message to the new template, diary_comment_notification.

First steps towards using layouts

Since all HTML mails will presumably share the same overall layout, and the markup is fairly dense, it makes sense to use Rails layouts to share the structure across mail templates. This commit moves the core structure of the HTML mail to a layout. It will need to be further refined so that notifications that involve a text message sent by another user can share the avatar-and-message-text structure.

Place GPS tiles in the overlayPane

Localisation updates from https://translatewiki.net.

Use https compatible names for GPS tile servers

Add public gps tiles layer for debugging purposes

Fixes #1239
Closes #1406

Remove HTML markup from locale file

Rather than including HTML markup in the locale file (which also has the inconvenience of requiring both a plain-text and an HTML duplicate of the same string), move the markup to the template. Also added a helper to reduce clutter in the template slightly.

More dependency tweaks for gem2.3 and apache2-dev

Closes #1405

ruby2.0->ruby 2.3

change dependancies. nokogiri requires a later ruby now as pointed out here: https://github.com/openstreetmap/openstreetmap-website/issues/1402 ruby2.0->ruby 2.3 ...which then entails a later ubuntu if we're installing from packages.

Switch vagrant Ubuntu Trusty -> Xenial

More compact image paths

Make the file paths to image assets more compact. I investigated using `image_path` but could only get it to return the path for a public URL, which is different and also includes the asset pipeline digest.

Make export action send TOTP cookie

Make export form send authenticity token

Proper way to get path to small avatar

Rather than performing perilous string substitution on paths, just use the built-in way to select the small version of the avatar image file.

Remove dot prefix from cookie domain

Reduce TOTP cookie expiry to one hour

Add support for generating TOTP cookies

This allows other sites in the openstreetmap.org domain to validate
that requests are coming from a www.openstreetmap.org user.

Localisation updates from https://translatewiki.net.

Fixed quotes

Replaced single quotes with double quotes, to comply with rubocop rules.

Remove unused variable

Use parameterised server root URL rather than hardcoded

Remove unused import

Move localised text to locale file

Moved a string of English text that will require translation from the HTML template to the locale I18N file.

Better loading of attached images

Got rid of that big ugly base64 blob which was only a temporary crutch anyway. Added a png file instead. Use File.read rather than File.open for a 66% reduction in verbosity.

Update bundle

Tweaked cell padding

Tweaked cell padding to align the top of the avatar and of the message text a bit better.

Cleaned up HTML

Removed some cargo-cult HTML attributes and CSS. Styling HTML for email clients is trickier than for browsers, I'm trying to keep the code as succinct but also as compatible as possible.

Fix clang build error

By default, Clang builds C code in GNU C11 mode, so it uses
standard C99 semantics for the inline keyword:

http://clang.llvm.org/compatibility.html#inline

Closes #1399

avatar inline attachment; text alignment

avatar inline attachment

reverting erroneous deletion in previous commit

intial commit, for 'en' locale only, for changeset_comment_notification only, doesn't load images properly yet

Add directions for straight on and slight left/right at roundabouts

Add directions for left/right turn at a roundabout

Localisation updates from https://translatewiki.net.

Update to iD v2.0.2

Localisation updates from https://translatewiki.net.

Use meta.wikimedia.org for wikimedia authentication

Merge remote-tracking branch 'openstreetmap/pull/1395'

Update OWG policy links.

Add support for Wikimedia authentication

Closes #1146

Localisation updates from https://translatewiki.net.

Load api fixtures in tag model tests

We still need them so we have the object to attach tags to.

Get rid of hardwired counts in assertions

Merge remote-tracking branch 'openstreetmap/pull/1367'

Fix misleading wording

We can't provide a free unlimited API for any third-party, not just developers. The change should further make translations less bumpy.

INSTALL.md grammar

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Don't collapse spaces in tag keys and values

Fixes #1386

Work around upcoming minitest insanity

Minitest 6 will not allow assert_equal to compare for equality
with nil and minitest 5 has already started warning about it.

That's fine if you're comparing with a nil constant, but if you're
comparing with an expression that is sometimes nil and sometimes
not nil it's an absolute pain in the rear end.

Fix rubocop warnings

Update bundle

Simplify monkey patch

The log method has grown new arguments which we weren't passing
on, but fortunately the exception translation is now delegated to
another method so we can patch that instead.

Remove redundant creation of an exception object

Make API and web roll back any open transactions on timeout

By default the exception thrown by Timeout::timeout is caught
using Kernel::catch so that it cannot be stopped by intermediate
exception handlers. The problem with that is that it stops any
database transactions that were in progress being rolled back
because they never see the exception.

Fortunately passing a class to Timeout::timeout changes it's
behaviour so that the exception is thrown and caught in the normal
way, allowing the database transactions to rollback.

Localisation updates from https://translatewiki.net.

Make sure email replies to diary comments come from the right user

Add tab icon for safari/touchbar

Closes #1382

Clone objects before saving in case of a retry

If a deadlock occurs then the transaction will be retried so we
need to make sure that the object will still be dirty so that it
will be saved again during the retry but that the version won't
be incremented a second time.

Localisation updates from https://translatewiki.net.

Increase password stretching to 10000 interations

This is in line with current reccomendations from various sources.

Use only_path instead of setting an explicit host

Use image_url instead of image_path with the host option

Fixes #1379

Add missing xmlns:dc for notes rss

Use the assets pipeline for images in the changeset feeds.

This brings it into line with the other feeds.

Use a full url for the image in the diary rss feed.

This is what was intended by ee2a941d3eb7279aed5829ddbafe1cc9f9631459
which fixed the same problem in a different place.

Add missing xmlns declaration for diary_entries

This is already present in the other feeds.

Localisation updates from https://translatewiki.net.

Update to leaflet 1.0.2

Merge remote-tracking branch 'openstreetmap/pull/1375'

Fix misspelling (lon -> lng)

Fixes https://github.com/openstreetmap/iD/issues/3588
Closes #1374

Update to iD v2.0.1

Add tags to features before propogating them.

Fix spelling of propagate.

Be explicit in which traces are expected.

I think this is an improvement compared to running db queries to
fetch the expected results.

Localisation updates from https://translatewiki.net.

Create object for list tests, and assert the responses have the
required number of results.

Previously, the tests would pass regardless of whether anything was
in the database.

Rename test trace fixtures to use letters instead of numbers.

This prevents them from being deleted by mistake, if trace.delete is
called on a factory-generated trace with a coincidental id.

Fix misspelling, @lng -> @lon

Update to iD v2.0.0