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

Allow blank issue templates

This was originally intended in #3397 and I'm not sure why I set this
to false.

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

Update bundle

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

Bump eslint from 8.6.0 to 8.7.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.6.0 to 8.7.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.6.0...v8.7.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.

Remove form_action restrictions for sessions#login

Login may redirect to ouath2_authorizations#create which may then
redirect to arbitrary schemes if the application is already authorized
so we need to allow login to redirect to any scheme.

Fixes #3424

Restore form_action restrictions for ouath2_authorizations#create

Remove form_action restrictions for ouath2_authorizations#create

Fixes #3424

Localisation updates from https://translatewiki.net.

rm spaces

add dir="auto" to search fields

Improves right-to-left text input support

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

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

Remove params from user deletion test

They have no effect, and are likely a copy-paste error from when
the test was first written in 39a54f8c14d739c95495e0d2c0ca56826eddfe52

Bump qs from 6.10.2 to 6.10.3

Bumps [qs](https://github.com/ljharb/qs) from 6.10.2 to 6.10.3.
- [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.10.2...v6.10.3)

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

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

Switch github URLs to use https

Fixes #3415

Avoid putting ActionController::Parameters objects in the session

Update bundle

Replace to_s on TimeWithZone objects with to_formatted_s

Localisation updates from https://translatewiki.net.

Update bundle

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

Use `assert_link` instead of `assert page.has_link?`

This leads to better error messages if the test fails

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

Use `assert_content` instead of `assert page.has_content?`

The assert_content comes from capybara, and gives a much more helpful
error message if the test fails.

Localisation updates from https://translatewiki.net.

Use factory_bot to build new model objects

Use factorybot to build user objects

This allows us to only specify attributes of interest in the test.

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

Attempt to avoid polynomial time matches on user supplied data

Fix display of suspension message when a user is suspended mid-session

Without the ability defined, the user is still logged out, but then
the deny_access check redirects to the login page. The re-login attempt
would then fail anyway, with an error message, but let's fix the abilities
and use the intended page.

Re-enable the Performance/StringIdentifierArgument cop

Update bundle

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

Remove redundant OpenID URL expansion code

It was only used for Google who have long since dropped OpenID support.

Bump eslint from 8.5.0 to 8.6.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.5.0 to 8.6.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.5.0...v8.6.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.

Switch to 6.1 defaults as everything has been enabled for some time

Localisation updates from https://translatewiki.net.

Test redirect from settings page to OmniAuth

Fix new rubocop warnings

Update bundle

Localisation updates from https://translatewiki.net.

Allow PATCH for OmniAuth requests

This is required to allow the account settings screen, which now
uses the PATCH verb, to redirect to OmniAuth when the external
authentication provider is changed.

As PATCH still uses CSRF this doesn't impact CVE-2015-9284 which
is the reason for requiring POST and most importantly got not
allowing GET requests to OmniAuth.

Localisation updates from https://translatewiki.net.

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

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

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

Remove unused require statement

The corresponding code was removed in a65cb8428867d92d76bbf051bbd4614966636cf5

Alias the user creation_time column

This allows rails to set the created_at automatically, and so avoids
us from having to do so in a callback. It also hides the unusual
db column name from the rest of the app.

Rename User#delete to User#destroy

"delete" is generally used for immediate SQL deletion without running
any callbacks or other ruby code, whereas "destroy" will trigger callbacks.

Although we don't currently use any callbacks, let's rename this method to
align better with the convention.

Update bundle

Localisation updates from https://translatewiki.net.

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

Bump eslint from 8.4.1 to 8.5.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.4.1 to 8.5.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.4.1...v8.5.0)

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

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

Update to rails 6.1.4.4

Localisation updates from https://translatewiki.net.

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

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

Update to rails 6.1.4.3

Update to rails 6.1.4.2

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Refactor the account edit/update pages out into a separate accounts controller

Bump eslint from 8.4.0 to 8.4.1

Bumps [eslint](https://github.com/eslint/eslint) from 8.4.0 to 8.4.1.
- [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.4.0...v8.4.1)

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

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

Update bundle

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

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

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

Bump qs from 6.10.1 to 6.10.2

Bumps [qs](https://github.com/ljharb/qs) from 6.10.1 to 6.10.2.
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/master/CHANGELOG.md)
- [Commits](https://github.com/ljharb/qs/compare/v6.10.1...v6.10.2)

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

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

Bump eslint from 8.3.0 to 8.4.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.3.0 to 8.4.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.3.0...v8.4.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.

Added `highway=turning_circle`

Localisation updates from https://translatewiki.net.

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

Revert to the last es5-compatible version of locatecontrol

Refs #3387

Merge pull request #3384 from HolgerJeromin/patch-1

bug issue template: removed browsers

Update bundle

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

Localisation updates from https://translatewiki.net.

Bump leaflet.locatecontrol from 0.74.1 to 0.76.0

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.74.1 to 0.76.0.
- [Release notes](https://github.com/domoritz/leaflet-locatecontrol/releases)
- [Changelog](https://github.com/domoritz/leaflet-locatecontrol/blob/gh-pages/CHANGELOG.md)
- [Commits](https://github.com/domoritz/leaflet-locatecontrol/compare/v0.74.1...v0.76.0)

---
updated-dependencies:
- dependency-name: leaflet.locatecontrol
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

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

remove browser list

bug issue template: resorting browsers

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

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

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

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

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

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

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

Localisation updates from https://translatewiki.net.

Drop the trace_use_job_queue option

This has been set as true by default, and in production, for many
years. I don't think there's much use in keeping the setting around
any longer.

Remove emergency=yes fallback description

This leads to confusion on features otherwise tagged (e.g. roads)
while not being necessary for nominatim results.

Fixes #3170

Prevent flash messages from expanding offscreen

Because we are using an edge-to-edge layout for the flash messages,
the negative margins on the row were expanding the element offscreen.

Bootstrap provides a "no-gutter" class for removing these margins,
but that also drops the spacing between columns which we want to preserve.
So instead we use a margin override on the row only.

Additionally, drop some padding css in favour of a utility class

Fixes #3351

Bug was introduced in 3dd639c8d02677b773935aa982ba0c9f3cc209e5

Add some issue templates and links to other repositories

This should help new users find the right place to go with their issue,
while also letting people use the blank template if they prefer.

The templates are heavily inspired by similar templates in the iD repo.

Fixes #3358

Use flex grid to position changeset discussion subscribe button

This avoids needing any floats. Also remove buttons class from two
locations, since there is a min-width on that class and it affects
the button styling negatively for standalone buttons

Remove some unused thumbnail rules

These are all done with flex grids now, so no need to float the image anywhere