[openstreetmap/openstreetmap-website] Use Noticed for notifications on GPX imports (PR #6939)

[Pablo Brasero]

pablobm left a comment (openstreetmap/openstreetmap-website#6939)

Good points, I should have stopped to think a bit more.

I had a look this morning, I see a couple of solutions:
- Have the `record` be the user instead of the trace.
	- This is what I've gone for at the moment, as it seemed simpler, but perhaps it's contrived.
- Use [ephemeral notifications](https://github.com/excid3/noticed/pull/382), which might be a more "pure" solution here. Notes:
	- With these, the initial delivery must be synchronous (no `deliver_later`). Note that the individual notifications (emails, etc) are still async/queued.
	- When it comes to on-site notifications, these would definitely require a separate model. That might be a good idea anyway; the proof of concept I put together last time (https://github.com/openstreetmap/openstreetmap-website/compare/master...pablobm:notifications) had plenty room for improvement.
	- I have experimented with this option at https://github.com/pablobm/openstreetmap-website/compare/convert-mailers-to-notifiers...pablobm:openstreetmap-website:ephemeral-notifications

Any preferences?

I have rewritten the PR description to reflect the latest changes, as well as answered the questions on specific changes.

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

Message ID: <openstreetmap/openstreetmap-website/pull/6939/c4134809252 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20260326/103feb8f/attachment.htm>