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

Fix some translation issues for diary_entry#new

Fixes #3880

Localisation updates from https://translatewiki.net.

Make sanitize convert text alignment styles to bootstrap classes

The kramdown converter for markdown uses inline styles for table
column alignment which gets lost when we strip the styles so add
bootstrap classes to replace it.

Merge pull request #3874 from tomhughes/offline

Allow models to load when the database is offline

Allow models to load when the database is offline

It used to be necessary to stop ActiveRecord loading because
it would immediately try and connect the database but with the
introduction of switchable database connections it now only connects
when a model is used for the first time.

Fixes #3858

Localisation updates from https://translatewiki.net.

Escape each portion of a semicolon seprated value individually

Fixes #3872

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

Update rubocop todo

Update bundle

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

Bump eslint from 8.30.0 to 8.31.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.30.0 to 8.31.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.30.0...v8.31.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.

fix formatting

Merge pull request #3870 from Dimitar5555/patch-1

Added missing `</p>` tag

Added missing `</p>` tag

fix formatting of tag values with semicolon-separated URLs

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

Add an entry for translation issues to the issue template

update-wiki-pages: process KeyPrefixDescription template

Update wiki_pages.yml

Preserve rel=me on links in rich text

Fixes #3859

Use CGI.escape for ruby 2.7 compatibility

Require integer IDs for diary_entries#show

Fixes #3865

Update bundle

Make test names clearer

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

Merge pull request #3864 from ZhengLin-Li/patch-1

Change Save to Comment

Change Save to Comment

For comment submit buttons we currently have:

"Comment" (Changeset Comments)
"Add Comment" (Issue Comments)
"Comment" (Note Comments)
"Save" (Diary Comments)

So, to make them consistent, I'd propose to change "Save" to "Comment"

Add tests to ensure tokens are revoked

This ensures that tokens are revoked or invalidated when a user
is soft destroyed.

update-wiki-pages: adding retry and timeout

Merge pull request #3860 from gravitystorm/yaml_quoting

Quote ruby versions in yaml

Quote ruby versions in yaml

Without quotes these are parsed as numbers, which can cause subtle
problems. For example, "ruby: 3.0" is interpreted by the
ruby/setup-rubyaction as "ruby: 3", meaning 3.x instead of the
intended 3.0.x

Localisation updates from https://translatewiki.net.

Don't allow any abilities for inactive users

Revoke authentication tokens when a user is deleted

Fix eslint warnings

Fix eslint warnings

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

consistent naming of FOSSGIS routers

added manoeuvre numbering

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

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

Localisation updates from https://translatewiki.net.

Use buttons for dropdown toggles

These elements don't link to another page, and so there's no need
to use a link that leads to nowhere.

Precompile assets before running tests in actions workflows

Make tmp persistent in docker

Update storage volume configuration for docker

Use rails instead of rake

Wait for the button to be disabled, before checking the tooltip behaviour

Rather than sleeping for a fixed period, we can use Capybara's built-in
ability to wait for elements to be present. Since we're only changing
the state of existing elements though, we need to wait for their
"disabled" property to be set.

Fixes some timing-related problems with the test suite.

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

Fix redirection to wiki when terms are declined

Adds missing test to pick this up.

Fixes #3826.