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

[Tobias]

@tordans commented on this pull request.

I don't know enough about he code but it looks like this likely duplicates some logic. Can this be shared in a concern or something similar?

> @@ -0,0 +1,152 @@
+# The NodeController is the RESTful interface to Node objects

Hint "NodeController"

> @@ -2,7 +2,7 @@ module Oauth
   SCOPES = %w[read_prefs write_prefs write_diary write_api read_gpx write_gpx write_notes].freeze
   PRIVILEGED_SCOPES = %w[read_email skip_authorization].freeze
   MODERATOR_SCOPES = %w[write_redactions].freeze
-  OAUTH2_SCOPES = %w[write_redactions openid].freeze
+  OAUTH2_SCOPES = %w[write_redactions message_send message_consume openid].freeze

Would it be more consistent to have something like 'write_messages', 'read_messages'?

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

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