[openstreetmap/openstreetmap-website] Add Messages API (PR #4605)

Milan Cvetkovic notifications at github.com
Thu May 30 08:21:08 UTC 2024


@milan-cvetkovic commented on this pull request.



> @@ -0,0 +1,10 @@
+json.id message.id
+json.from_user_id message.from_user_id
+json.to_user_id message.to_user_id

User names are not available in message itself, I would have to either modify the model and include those two, or query for user names in order to return them.

They would be nice to have, but they would cost some database lookups

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4605#discussion_r1620233739
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4605/review/2087588757 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240530/d1a7d3b2/attachment.htm>


More information about the rails-dev mailing list