gravitystorm left a comment (openstreetmap/openstreetmap-website#6791)
> I'd like to work on this! It seems like a straightforward update to the
> welcome banner. I can prepare a PR to add the new string to en.yml and update
> map.html.erb as suggested.
>
> Since I'm currently having some issues with my local Docker environment, I'll
> focus on the code implementation and ensuring the logic matches the
> requirements mentioned by [@1ec5](https://github.com/1ec5).
If you are planning on making changes to the UI, but you aren't actually able
to view these changes locally, then please stop and rethink what you are doing.
You've already had one PR declined because you didn't review the changes
yourself. I appreciate that you are willing to help, but writing code without
checking that it works isn't actually helpful at all.
If you are having problems with your local Docker environment, please fix that
first. If you are stuck, feel free to open an issue describing the problem, and
hopefully we can help!
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/6791#issuecomment-3892284917
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/issues/6791/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
