Re: [openstreetmap/openstreetmap-website] Use map tiles dark mode from leaflet-osm plugin (PR #5426)

2025-01-17 Thread Marwin Hochfelsner via rails-dev
Closed #5426. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5426#event-15985051278 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-dev mailing

[openstreetmap/openstreetmap-website] Remove public domain checkbox from signup and terms pages (PR #5514)

2025-01-17 Thread Anton Khorev via rails-dev
You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5514 -- Commit Summary -- * Add empty pd declaration page * Show checkbox on pd declaration page * Add explanation link to pd declaration page * Add confirm butt

[openstreetmap/openstreetmap-website] Remove "Public editing" line from settings page (PR #5513)

2025-01-17 Thread Anton Khorev via rails-dev
You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5513 -- Commit Summary -- * Remove "Public editing" line from settings page -- File Changes -- M app/views/accounts/edit.html.erb (13) M config/locales/en.ym

[openstreetmap/openstreetmap-website] Friend cleanups (PR #5512)

2025-01-17 Thread Tom Hughes via rails-dev
This continues the renaming of friends as follows by tidying up some of the remaining code references to friendships. You can view, comment on, or merge this pull request online at: https://github.com/openstreetmap/openstreetmap-website/pull/5512 -- Commit Summary -- * Rename friends_with t

Re: [openstreetmap/openstreetmap-website] Use resourceful route for user status (PR #5437)

2025-01-17 Thread Andy Allan via rails-dev
Merged, thanks! -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5437#issuecomment-2598896243 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-dev

Re: [openstreetmap/openstreetmap-website] Use resourceful route for user status (PR #5437)

2025-01-17 Thread Andy Allan via rails-dev
Merged #5437 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5437#event-15980674827 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Fix missing translation of Follow links on dashboard (PR #5508)

2025-01-17 Thread Andy Allan via rails-dev
Merged #5508 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5508#event-15980663813 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Fix missing translation of Follow links on dashboard (PR #5508)

2025-01-17 Thread Andy Allan via rails-dev
Merged, thanks! -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5508#issuecomment-2598894180 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-dev

Re: [openstreetmap/openstreetmap-website] Use form with normal-sized button on follows show pages (PR #5509)

2025-01-17 Thread Andy Allan via rails-dev
Merged #5509 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5509#event-15980656428 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-

Re: [openstreetmap/openstreetmap-website] Use form with normal-sized button on follows show pages (PR #5509)

2025-01-17 Thread Andy Allan via rails-dev
Merged, thanks! -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5509#issuecomment-2598893179 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-dev

Re: [openstreetmap/openstreetmap-website] Use dark map tiles classlessly (PR #5507)

2025-01-17 Thread Marwin Hochfelsner via rails-dev
@hlfan commented on this pull request. > @@ -9,7 +9,7 @@ body { font-size: $typeheight; - --dark-mode-map-filter: brightness(.8); + --dark-mode-map-filter: none; Having the custom property defined with a default value of `none` increases compatibility, since it doesn't even alter trans

Re: [openstreetmap/openstreetmap-website] Add OAuth Refresh token when `openid` scope is used (PR #5497)

2025-01-17 Thread mmd via rails-dev
I'm wondering why you want to send id_tokens to a resource server in the first place. That's not really their intended use: https://oauth.net/id-tokens-vs-access-tokens/ -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5497#issuec

[openstreetmap/openstreetmap-website] Adds optional use of notes records (PR #5511)

2025-01-17 Thread Nenad Vujicic via rails-dev
### Description PR adds use of `description`, `user_id`, `user_ip` from `notes` table if data-migration is done, otherwise use `body`, `author_id`, `author_ip` from note's first comment. Decision procedure "data-migration is done" is defined with "is user_ip or user_id not nil?". This PR is 4t

[openstreetmap/openstreetmap-website] Adds validating of notes foreign key (PR #5510)

2025-01-17 Thread Nenad Vujicic via rails-dev
### Description PR adds migration for validation foreign key in `notes` table referencing `users` table. This PR is 3rd from set of PRs described [here](https://github.com/openstreetmap/openstreetmap-website/pull/5485#pullrequestreview-2545467592). ### How has this been tested? Automated unit