[openstreetmap/openstreetmap-website] On-site notifications (PR #7030)

[Pablo Brasero]

@pablobm commented on this pull request.



> +            t(".changeset_comment"),
+            changeset_path(
+              notification.changeset,
+              :anchor => "c#{notification.comment_id}"
+            )
+          ),
+          :commenter_name_link => link_to(
+            notification.commenter.display_name,
+            notification.commenter
+          ),
+          :time_ago => friendly_date_ago(notification.timestamp)
+        ) %>
+  </p>
+  <blockquote class="mx-2 fst-italic">
+    <p class="text-truncate"><%= notification.comment_body.truncate(200, :separator => " ") %></p>
+  </blockquote>

Yes, it makes sense to provide a unified "take me to the source of the notification" button.

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

Message ID: <openstreetmap/openstreetmap-website/pull/7030/review/4168002337 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260423/197665be/attachment.htm>