[openstreetmap/openstreetmap-website] Fix new Herb offenses (PR #6958)

Marwin Hochfelsner notifications at github.com
Thu Apr 2 11:48:50 UTC 2026


@hlfan commented on this pull request.



> @@ -98,7 +98,7 @@ def check_trace_feed(traces)
             assert_select item, "guid", "http://www.example.com/user/#{ERB::Util.u(trace.user.display_name)}/traces/#{trace.id}"
             assert_select item, "description" do
               assert_dom_encoded do
-                assert_select "img[src='#{trace_icon_url trace.user, trace}']"
+                assert_select "img[src='#{trace_icon_path trace.user, trace}']"

@gravitystorm proposed that originally, so he may know more.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6958?email_source=notifications&email_token=AAK2OLKGHMOUPTXLRB2WZST4TZHSFA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBVGAYTGOJRGM4KM4TFMFZW63VKON2WE43DOJUWEZLEUVSXMZLOOS6XA4S7OJSXM2LFO5PW433UNFTGSY3BORUW63TTL5RWY2LDNM#discussion_r3027574512
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/6958/review/4050139138 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260402/ab167262/attachment.htm>


More information about the rails-dev mailing list