Re: [openstreetmap/openstreetmap-website] Support interactive debugger with docker-compose (PR #5851)

2025-06-20 Thread rkoeze via rails-dev
@rkoeze pushed 1 commit. cc4d613085bee5e7fafe78a9cab8cdfd8ac335e7 Support interactive debugger -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5851/files/45b947f4a9661af57e25567fc2db821c2bbf287b..cc4d613085bee5e7fafe78a9cab8cdfd8ac335e7 You are receiving this b

Re: [openstreetmap/openstreetmap-website] Add recent diaries partial to profile page (PR #6010)

2025-05-14 Thread rkoeze via rails-dev
@rkoeze commented on this pull request. Looks great. Left a couple of comments. I wonder if there's an opportunity to use some of these design elements on the `/diaries` page. > @@ -0,0 +1,27 @@ +<% if diary_entries.present? %> + <%= t(".recent_diaries") %> Would it make sense to style the "R

Re: [openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

2025-05-14 Thread rkoeze via rails-dev
@rkoeze pushed 2 commits. 8fadbd353415cb69986d662aa368a4dc8ac33a53 Move gravatar_enable method to User model 3b7dce75b2d9ce4f27d163b3bd4383c6da1698ea Bump ClassLength rubocop_todo -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5988/files/d8423e8e10091dfe6ce9

Re: [openstreetmap/openstreetmap-website] Support interactive debugger with docker-compose (PR #5851)

2025-05-14 Thread rkoeze via rails-dev
@rkoeze pushed 1 commit. 45b947f4a9661af57e25567fc2db821c2bbf287b Support interactive debugger -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5851/files/75d3fed09441b7e7cb8b2cad424ccdea024f92e6..45b947f4a9661af57e25567fc2db821c2bbf287b You are receiving this b

Re: [openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

2025-05-09 Thread rkoeze via rails-dev
@rkoeze pushed 1 commit. d8423e8e10091dfe6ce9b3fd78105b92f7c23ee3 Bump ClassLength rubocop_todo -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/5988/files/8fe268d7e1eaedc4b7c1028a2043b062b83ffe74..d8423e8e10091dfe6ce9b3fd78105b92f7c23ee3 You are receiving this

Re: [openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

2025-05-07 Thread rkoeze via rails-dev
rkoeze left a comment (openstreetmap/openstreetmap-website#5988) As a fly-by-night contributor I'll happily defer to whatever course of action you prefer. A couple of thoughts: - I put together [a quick spike](https://github.com/rkoeze/openstreetmap-website/pull/2) showing how we might use conc

Re: [openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

2025-05-06 Thread rkoeze via rails-dev
rkoeze left a comment (openstreetmap/openstreetmap-website#5988) It looks like I'm bumping up against the `ClassLength` cop [here](https://github.com/openstreetmap/openstreetmap-website/blob/master/.rubocop_todo.yml#L68). Happy to close this PR if that's a strict rule we need to observe. We coul

[openstreetmap/openstreetmap-website] Move `gravatar_enable` method to User model (PR #5988)

2025-05-06 Thread rkoeze via rails-dev