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

Bump qs from 6.11.0 to 6.11.1

Bumps [qs](https://github.com/ljharb/qs) from 6.11.0 to 6.11.1.
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](https://github.com/ljharb/qs/compare/v6.11.0...v6.11.1)

---
updated-dependencies:
- dependency-name: qs
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

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

Update to iD v2.25.1

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

Localisation updates from https://translatewiki.net.

Bump coverallsapp/github-action from 1.2.2 to 1.2.4

Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.2.2 to 1.2.4.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](https://github.com/coverallsapp/github-action/compare/v1.2.2...v1.2.4)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix new rubocop warnings

Update bundle

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

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

Remove duplicate translation key

It's already defined three lines above.

Remove old model name

I doubt this was ever used, but anyway it was made redundant in
4395b79bf72f85543bce8ff3627eb7a0583b5ccc when the class was renamed.

Bump coverallsapp/github-action from 1.2.1 to 1.2.2

Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.2.1 to 1.2.2.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](https://github.com/coverallsapp/github-action/compare/v1.2.1...v1.2.2)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump eslint from 8.34.0 to 8.35.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.34.0 to 8.35.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.34.0...v8.35.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Remove translations that are no longer used when showing messages

Remove used translations for new messages

We use the activerecord attribute hierarchy for this

Remove unused translations

These were removed in 6adcce4e5d75fa21dea85742fa3

Bump coverallsapp/github-action to 1.2.1

Localisation updates from https://translatewiki.net.

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

Remove unused tooltip translations

These were removed in 2d9a083b365e5cfa527c40add35464630c1a9b00

Remove unused metro extract translations

Refs 9340c461732ef7387dd453ea3fcf5e64ea782ab4

Remove translations no longer used on the login page

This page was refactored in 2a3bc0a38789b29b8798eafd80fc4fb77c5c7957

Bump coverallsapp/github-action from 1.1.3 to 1.2.0

Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 1.1.3 to 1.2.0.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](https://github.com/coverallsapp/github-action/compare/1.1.3...v1.2.0)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

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

Merge pull request #3935 from Hufkratzer/rename-horse-riding-label

Change label for leisure=horse_riding in en.yml

Rework about page to avoid html in the translations

Remove html from translations for the welcome page

This also refactors the "any questions" section, which is shared with the fixthemap page.

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

Ensure kramdown table alignment styles are converted to bootstrap classes

Adds tests for commit f42a7a2d8dd507a3746a1e2de91deda9614cf807

Refactor trace guidance to avoid raw html in translations

Also rearrange to avoid multiple sentences in a heading.

Refactor translation to avoid raw html tags

Move public editing content into its own partial

This makes the editing page easier to read, and simplifies the i18n
in the public editing section.

Refactor public editing note to avoid raw html in the translations

Update bundle

Localisation updates from https://translatewiki.net.

Change label for leisure=horse_riding in en.yml

Change "Horse Riding" to "Horse Riding Centre"

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

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

Bump eslint from 8.33.0 to 8.34.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.33.0 to 8.34.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.33.0...v8.34.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

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

Localisation updates from https://translatewiki.net.

Update to iD v2.24.2

Merge pull request #3691 from AntonKhorev/wikipedia-secondary-links

Link prefixed wikipedia tags

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

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

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

Avoid converting conditional restriction tags into emails

Fixes #3884

Refactor to use submit_tag helper

This is more readable than interpolating into element attributes

Refactor translation strings for notes

These aren't used by any javascript code, so don't need to be in
the `javascripts` key.

Use data attributes to pass alternative button labels

This allows the text to be defined in the html partials, instead
of looking up the i18n via javascript.

Move browse#note to notes#show

This allows a more resourceful routing approach.

Move browse#new_note to notes#new

This allows a more resourceful routing approach.

Update bundle

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

Bump eslint from 8.32.0 to 8.33.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.32.0 to 8.33.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.32.0...v8.33.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

Update to rails 7.0.4.2

Localisation updates from https://translatewiki.net.

Sort issues in descending order of their last update

Rename api_notes resource to avoid conflict with non-API resource

This allows using e.g. note_path for notes#show in the future.

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

Update to iD v2.24.1

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

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Update to iD v2.24.0

Localisation updates from https://translatewiki.net.

Update to rails 7.0.4.1

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

Update bundle

Bump eslint from 8.31.0 to 8.32.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.31.0 to 8.32.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.31.0...v8.32.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

Update tests for changes in osm-community-index

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

Bump osm-community-index from 5.4.0 to 5.5.0

Bumps [osm-community-index](https://github.com/osmlab/osm-community-index) from 5.4.0 to 5.5.0.
- [Release notes](https://github.com/osmlab/osm-community-index/releases)
- [Changelog](https://github.com/osmlab/osm-community-index/blob/main/CHANGELOG.md)
- [Commits](https://github.com/osmlab/osm-community-index/compare/v5.4.0...v5.5.0)

---
updated-dependencies:
- dependency-name: osm-community-index
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Drop version constraint on mail and update bundle

Localisation updates from https://translatewiki.net.

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

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

Merge pull request #3878 from tomhughes/issues-offline

Add database checks to issue and report controllers

Move more api-related helper methods to ApiController

This prevents them from being inadvertently used in non-API controllers

Use web_timeout for notes controller

This controller doesn't handle any API methods, and so the api call
error handling isn't appropriate.

Use bin/yarn consistently for installations

Since rails 7, the `rails yarn:install` command no longer uses
bin/yarn and so a) doesn't detect yarn binaries named "yarnpkg"
(e.g. Ubuntu) and b) doesn't allow us to pass any flags to yarn,
(e.g. "--ignore-engines").

Since we want to avoid making the installation more difficult (by
e.g. installing yarn from other sources, or requiring particular
node versions) we can instead just use our existing bin/yarn script.

The reason Dockerfile did its own thing was to avoid pulling in
big chunks of rails, but bin/yarn is a standalone ruby script.

Fixes #3579

Add database checks to issue and report controllers

Fixes #3875

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

Merge pull request #3889 from tomhughes/distances

Make distances in directions translatable

Make distances in directions translatable

Fixes #3888

Update bundle

Remove Geonames and geocoder.ca

Make error pages translatable

Fixes #3877

Merge pull request #3885 from tomhughes/diary-new

Fix some translation issues for diary_entry#new