Put the expected value first in assert_equal

Move controller tests for new reports into seperate file, and adapt to new form structure.

Index page no longer redirects when there's no issues.

Move and amend the issue comment controller tests.

Remove column sorting

It's not tested, not working, and so it can come later in a separate PR.

Rubocop fixes.

Remove the reassign boolean display

Use a simple user_id for tables with one owning user

This is the standard way of naming in rails, and it avoids having
to tell rails which models are actually required.

Combine migrations into one

This makes it easier to review, rather than having a PR with migrations that
correct each other.

Remove the reassign column from issue_comments

Reassigning is an effect of a controller action, not something that
should be stored in the database like this.

Adjust for rails5 convention on migrations.

Annotate models.

Refactor factories to use FactoryBot

Remove minitest-rails-capybara and convert feature tests to system tests

Merge branch 'master' into moderation

Localisation updates from https://translatewiki.net.

Extend form-action policies for Chrome

It seems that unlike other browsers Chrome requires that if a form
submission redirects that the redirected URL also match the form-action
policy rather than just requiring the original URL to match.

Remove inline javascript from user settings page

Allow iD to fetch gpx files from arbitrary locations

Fix security policy for mapillary in iD

Fix remote editing security policy

Add security policy for remote control editing

Allow iD to connect to nominatim

Allow third party images in user content

Allow images to be loaded from piwik

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

Update CONTRIBUTING to remove reference to cruise and use link to Travis

Localisation updates from https://translatewiki.net.

Add the OpenStreetMap Blog to the list of community links

Although the content is currently duplicated into the blog aggregator,
it's still worth having an explicit link, and making it the first one,
since it contains entries of more significant interest to the community.

Put specific page titles at the start

The makes the shortened titles commonly shown on browser tabs more
useful.

Merge branch 'pull/843' into titles

Merge branch 'p' of https://github.com/jfirebaugh/openstreetmap-website into jfirebaugh-p

Refs #139

Localisation updates from https://translatewiki.net.

Make context menu use from/to for directions

This ensures that the router will try to geocode the values rather
than assuming that they are latitude/longitude pairs.

Fixes #1684

Add openstreetcam.org to security policy for iD

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

Localisation updates from https://translatewiki.net.

Update to iD v2.5.1

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.