[openstreetmap/openstreetmap-website] Add Messages API (PR #4605)
Milan Cvetkovic
notifications at github.com
Fri Mar 22 17:48:59 UTC 2024
@milan-cvetkovic commented on this pull request.
> +
+ # Return a copy of the message
+ respond_to do |format|
+ format.xml { render :action => :show }
+ format.json { render :action => :show }
+ end
+ end
+
+ private
+
+ def show_messages
+ if params[:min_id]
+ @messages = @messages.where("id >= ?", params[:min_id]) unless params[:min_id] == "0"
+ @messages = @messages.order(:id => :asc)
+ else
+ @messages = @messages.where("id <= ?", params[:max_id]) unless params[:max_id].nil? || params[:max_id] == "0"
Modified to use `from_id` instead of `min_id` and `max_id`, and use absence of `from_id` to retrieve the first batch.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4605#discussion_r1535974592
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/4605/review/1955419936 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20240322/f1deeb9b/attachment.htm>
More information about the rails-dev
mailing list