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

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

Merge branch 'pull/5609'

Remove spaces before colons in en.yml

Update to iD v2.31.1

Localisation updates from https://translatewiki.net.

Removes dropping note's first comment

Removes dropping note's first visible comment in case of deleted note's author. After adding displaying "deleted" as note's description, first visible comment is now displayed as note's comment. Moves logic of calculating which note comments will be displayed and if note contains anonymous author / comments to the controller.

Switches from comments to all_comments

Switches from using note's .comments to .all_comments. Fixes bug with notes of deleted users without comments, which are filtered out, but the code tries to access the first comment.

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

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

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

Simplify initialisation of iD

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

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

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

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

Map 'full' to api relation show action

Map 'full' to api way show action

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

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

Merge pull request #5617 from AntonKhorev/api-element-resources--show-paths

API element resources - show/update/delete paths

Test Last-Modified header of api show element responses

Test if api relation show/full responses contain required elements

Split api node show test

Split api way show test

Split api relation show/full tests

Move api element show and full tests

Make api show/update/destroy relation actions resourceful

Make api show/update/destroy way actions resourceful

Make api show/update/destroy node actions resourceful

Image lazy loading

Fixes #5601

Merge pull request #5591 from AntonKhorev/api-element-resources--index-paths

API element resources - index paths

Merge pull request #5559 from hlfan/eslint-plugin-erb

Add eslint-plugin-erb to provide linting of .js.erb files

Merge pull request #5608 from AntonKhorev/reassign-issue-name-queue

Write the new role in reassign checkbox label

Replace deprecated eslint rules

no-new-object => no-object-constructor
no-process-env => will be catched by no-undef

Merge pull request #5606 from AntonKhorev/no-eslint-qs

Remove Qs from eslint globals

Add eslint-plugin-erb for .js.erb file linting

* Add eslint-plugin-erb to provide linting of our .js.erb files
* Lint osm.js.erb and API-ify OSM.params()
---------
Co-authored-by: Andy Allan <git@gravitystorm.co.uk>

Merge branch 'pull/5557'

Move deprecated eslint rules to stylistic plugin

Update to iD v2.31.0

Write the new role in reassign checkbox label

Remove Qs from eslint globals

Reevaluate iteration methods

Don't html-escape note description

The description is already html-safe after to_html.

Merge branch 'pull/5579'

Test if closing sidebar removes directions popup

Close directions popup with sidebar

Adds optional use of note's author and description

Adds optional use of author and description from notes (in case data-migration is done) and author and description from first visible note comment (in case data-migration is not done). Also, adds author association to Note model.

Removes note's author from RSS

Removes writing note's author to note's RSS if author is missing or data-migration is done and note's author is deleted.

Replaces using description with note_description

Replaces using note's description method with note_description helper routine.

Adds helper routine note_description

Adds new helper routine note_description for retrieving note's description. Helper routine returns "deleted" if author is deleted otherwise note's description.

Fix missing zoom argument of OSM.cropLocation in home location

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

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

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

Update bundle

Refactor api changeset discussion tests

Don't apply extra encodeURIComponent to lat/lon search params

Merge branch 'pull/5592'

Merge branch 'pull/5594'

Replace qs with URLSearchParams

Change "reporter users" to "reporting users"

Don't check ids of new notes in create note system tests

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

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

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

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

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

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

Make api create relation path resourceful

Make api create way path resourceful

Make api create node path resourceful

Move api element index tests up

Make api element index paths resourceful

Add layer-type switching to use dark tiles

Add coordinate precision function

Use URI.encode_uri_component when generating share urls

Use flat_map to collect way node ids in maps controller

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

Use filter instead of loop in visible node/way collections

Remove ids collected in parallel to model instances in map api

Remove unused visible_nodes hashes from api element controllers

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

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Refactor embed.js.erb with layers.yml

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

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

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

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

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

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

Select user ids from reports using pluck

Merge branch 'pull/4990'

Abort loading when note layer is removed

Show only description as a marker tooltip in note layer

Abort loading when data layer is removed

Remove unnecessary wrapper divs from data layer error message

Don't remove-readd standard layer when layerParam is not empty in map.updateLayers