Make system tests run by default

Add framework for system tests

Merge branch 'wheres_this' of https://github.com/pnorman/openstreetmap-website into pnorman-wheres_this

Fixes #412

Localisation updates from https://translatewiki.net.

Add iD disable_features parameter

Update to iD v2.5.0

Remove link to wiki legal FAQ as confusing and unmaintained

Disable keyboard controls for layer switcher maps

Fixes #1673

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

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Update site to use jQuery 3

When no cache store is set use cookie storage for sessions

A change in the default configuration of development environments
in rails 5.1 means that caching is now disabled by default so in
that case fall back to using cookies instead.

Closes #1666

Localisation updates from https://translatewiki.net.

Change wording for hosting partners

Relax cookie security policy

Localisation updates from https://translatewiki.net.

Fix rubocop warnings

Update bundle

Replace factory_girl with factory_bot

Avoid using rinku's auto_link helper

It's currently broken (https://github.com/vmg/rinku/issues/70) and
in case our own linkify has some additional customisations.

Update for ActionView::Template::Error change

Localisation updates from https://translatewiki.net.

Convert session timeout to seconds when it is set

This avoids problems if the rails Duration class changes
representation between different versions of rails.

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

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

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

Localisation updates from https://translatewiki.net.

Rubocop fixes

Add annotate gem and associated rake task. Annotate models.

Avoid using or comparing explicit model ids

The code is easier to read using higher-level concepts.

Localisation updates from https://translatewiki.net.

Removes DOM difference in location button (main map)

Removes DOM difference in location button

Add rails version number to migrations

Update to rails 5.1.4

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

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

Localisation updates from https://translatewiki.net.

Add banner for OSMF Membership Drive

Localisation updates from https://translatewiki.net.

Update to iD v2.4.3

Use send_data for GPX traces intead of monkey patching send_file

Improve user_image_url test

Use a user that actually has an image and updated the expect
output to match that.

Remove reference to large screen stylesheet

The separate large screen stylesheet was removed in 4dceacdeb but
we are still referencing it on Internet Explorer.

Fix typo that means fe0f4543c1 has never worked

Update to iD v2.4.2

Localisation updates from https://translatewiki.net.

Use Hash#dig to simplify code

Repair bundle

Fix rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Drop geocoder.us from search engines

Fixes #1633

Add banner-sotmus2017.png

Delete banner-sotm-2017.png

Add SOTM US and remove SOTM.org

Defer measuring of header elements to a timeout

Chrome 60 and later seem to fire the "ready" callback before the DOM is
fully ready causing us to measure the wrong sizes for the header elements
so we use a 0ms timeout to defer the measurement slightly as a workaround.

Fixes #1639

Remove some more of the structured-details stuff

Create an IssueCommentsController for managing IssueComments

Fix parameter spacing

Rework the 'issues not found' notice slightly.

Previously it would go into a redirect loop if there were no issues
in the database.

The link is to the list of issues, not of reports, and there's no need for it to be bold.

Ensure that issue comments have associated issues and users.

Add missing structure.sql changes from 948fd8c8faf1016d

Remove created_at / updated_at column definitions since these duplicate t.timestamps

Add banner for State of the Map Asia 2017

Closes #1630

Add padding between the last diary post and the pagination links

Closes #1634

Update secure_headers configuration for upstream changes

Update travis configuration to work on trusty based builders

Switch to postgres 9.5 and link postgres functions to a directory
that is accessible to the postgres server.

Update rails to 5.0.6

Ensure report details are not blank.

Refactor creating a new report to use a ReportsController

It's really a report that the user is creating, the issue is created indirectly. Similar
refactoring will be required for issue comments.

Use reports_count so that counter_cache works automatically.

Remove the border around the radio boxes

It looks strange.

Check that the flag is not show when not logged in

Add a basic feature test for reporting diary entries.

Add minitest-rails-capybara

This allows us to write feature tests using capybara. These can be
easily changed to system tests when we upgrade to rails 5.1

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

Rename xml processing methods with an update_ prefix

They behave differently from the other from_xml methods on other models.

Enable retina tiles for Thunderforest layers

Fixes #1623

Update to iD v2.4.1

Add iD hashtags parameter

Update to iD v2.4.0

Refactor the from_xml methods to act on existing trace objects.

Setting the new tags with the = operator takes care of removing the
old ones, and is the same approach as taken by the tagstring= method.

Fixes #1600

Add failing test demonstrating tag duplication when updating traces via the API.

Make sure we show errors correctly when resetting a password

Discard empty ways from query result geometries

Fixes #1608

Update to leaflet 1.2.0

Fix rubocop warning

Switch to using openstreetmap-deadlock_retry

Closes #1602

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

Add maintainers to README

It's nice to be explicit, so that new contributers know who is who.

While github shows "owner" and "member" labels these are for the
openstreetmap organisation as a whole, and can be misleading.

Make sure the account form is POSTed

Fixes #1601

Merge pull request #1588 from Georeactor/master

Update gem, fix icons on RTL/Arabic pages

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

Avoid passing the current_user.id when commenting

Doing so doesn't appear to be necessary, and it opens up a spoofing
hole if it's actually used.

Remove unused user save template.

This template isn't used, since all the controller methods redirect
to other views.

Use current_user instead of @user.

Merge branch 'master' into moderation