Added description, user_id, user_ip columns to notes

Added migration for adding new columns (description, user_id, user_ip) to notes table. Also, migration adds foreign key connecting notes and users tables (using user_id column).

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

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

Rename gpx description template to details

"Description" is just one of trace properties, details include description and other things.

Move number of points to file details in gpx emails

Bump eslint from 9.17.0 to 9.18.0

Bumps [eslint](https://github.com/eslint/eslint) from 9.17.0 to 9.18.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/v9.17.0...v9.18.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/5487'

Merge branch 'pull/5484'

Show diary entry after creation

This allows users to review their post and use the social share
buttons more easily.

Refs #1391

Move trace descriptions out of success/failure sentences in emails

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

Merge pull request #5462 from AntonKhorev/user-terms-resource

Use resourceful routes for terms view/accept/decline

Use resourceful routes for terms view/accept/decline

Update documentation to avoid erb_lint deprecations

I didn't spot this in #5357

Remove slashes from body css classes

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

Restrict version of dry-schema for ruby 3.1.2 compatibility

https://bugs.ruby-lang.org/issues/18673 affects some methods using
anonymous blocks, and was fixed in ruby 3.1.3. However, debian/bookworm
ruby 3.1 package is based on 3.1.2 and does not yet contain this fix.

Since bookworm is used in our Docker build, and also used in production
by OSMF, then we can restrict dry-schema to a version that does not
use anonymous blocks.

Fixes #5482

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

Bump leaflet.locatecontrol from 0.83.0 to 0.83.1

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.83.0 to 0.83.1.
- [Changelog](https://github.com/domoritz/leaflet-locatecontrol/blob/gh-pages/CHANGELOG.md)
- [Commits](https://github.com/domoritz/leaflet-locatecontrol/compare/v0.83.0...v0.83.1)

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

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

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

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

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

Update bundle

Move browse/feature template to elements/show

Add ElementsController base class

Report deactivates_at as block read date

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

Update redefinition of interestingNode to match leaflet-osm

Update arguments of data overlay interestingNode()

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

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

Check deactivates_at to report if block was read after its end date

Make browse not found pages consistent

Update leaflet-osm to latest upstream version

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

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

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

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

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

Use details attribute to render changeset bounding box

Do this instead of using L.OSM.DataLayer. DataLayer makes an extra request just to get the bbox. Also it expects bbox to always exist, which is not true for some changesets.

Write details attribute to changeset page

Move changeset data to a helper

Remove 'center' argument of addChangeset()

It is always set to true.

Remove zoomend listener on history unload

Allow thirdparty images on failed diary comment saves

Fix options passed by allow_thirdparty_images

Show outbox after sending new message

Move redacted version locale strings to old_elements

Move history_title_html locale string to old_elements

Move browse history template to old elements directory

Move element version actions template to old elements directory

Add OldElementsController base class

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

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

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

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

Rename Account module to Accounts

Convert Auth.providers into array

Use resourceful routes for user new/create actions

Remove "expired unread" short block status

It was supposed to mean that the block is past its end date but still wasn't read by the user. That turned out to be confusing because although it said "expired", such blocks were active.

Use resourceful route for message reply

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

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

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

Localisation updates from https://translatewiki.net.

Use resourceful route for new message

Remove test_logout_removes_session_token

Remove commit parameter from diary controller test

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

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

Move new note marker to click location

If not done, it's possible to lose the marker when starting from low zooms.

Decrease new note marker opacity when zoomed out

Let page.load proceed when zoomed out, disabling submit

Note that addNoteButton.hasClass("active") check in page.load is useless because page.unload removes this class.

Remove active class from addNoteButton only in page.unload()

Same reasons as in the previous commit.

Call removeNewNoteMarker() only from page.unload()

It was also called from noteCreated() but that triggered page.unload() via OSM.router.route().

Add addNewNoteMarker() and removeNewNoteMarker() functions

This ensures that marker is always added/removed together with its halo.

Convert newHalo() to addHalo() and removeHalo()

newHalo() wasn't always creating new halos. It had an event type argument that was checked against drag event. Depending on this check, newHalo() could remove the halo.

Move new note page.pushstate/popstate next to page.load

Add new note marker to root map layer instead of noteLayer

We want the new note marker in the same layer with its halo. Also we don't want to put anything in noteLayer because it clears its contents on certain zoom levels.

Remove unused return value of addCreatedNoteMarker()

This function adds a marker to notes layer. This controller doesn't need to do anything with markers of existing notes.

Rename updateMarker() to addCreatedNoteMarker()

This function doesn't update any existing marker.

Pass location instead of marker to createNote()

Move create note callback to button click handler

Don't pass url to createNote()

Pass text instead of form to createNote()

Disable new note submit button in click handler

Rename newNote to newNoteMarker

Update bundle

Bump leaflet.locatecontrol from 0.82.0 to 0.83.0

Bumps [leaflet.locatecontrol](https://github.com/domoritz/leaflet-locatecontrol) from 0.82.0 to 0.83.0.
- [Changelog](https://github.com/domoritz/leaflet-locatecontrol/blob/gh-pages/CHANGELOG.md)
- [Commits](https://github.com/domoritz/leaflet-locatecontrol/compare/v0.82.0...v0.83.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>

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

Update leaflet-osm to latest upstream version

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

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

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

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

Use resourceful routes for web mailboxes

Replace resent confirmation link with button

Move code that runs after user save from save_new_user to create method

Test signup with confirmation email

Add within_content_body system test helper