Ticket #742 (closed enhancement: fixed)

Opened 6 years ago

Last modified 10 months ago

API lacks Notes/Feedback system

Reported by: openstreetmap@… Owned by: rails-dev@…
Priority: minor Milestone: Wishlist
Component: api Version:
Keywords: notes, feedback Cc:

Description

The  OSM_Notes_API is missing from the main OSM API.

Attachments

rails_port_notes.diff Download (12.4 KB) - added by openstreetmap@… 6 years ago.
A first cut patch to implement the missing functionality.
rails_port_gpx.diff Download (2.9 KB) - added by openstreetmap@… 6 years ago.
Patch to pull notes from uploaded gpx files (untested due to rmagick install/version issues plus gps_points has no id field apparently needed by trace.rb#186 exists?)

Change History

Changed 6 years ago by openstreetmap@…

A first cut patch to implement the missing functionality.

comment:1 Changed 6 years ago by tom@…

Thanks for the submission - I'll have a look at it in detail after Easter (I'm away until then).

I would just say that it might have been a good idea to say what you were doing, as I know there are other people that have been looking at this...

It looks OK from a quick scan, although it looks like it doesn't actually interface with the web site at all at the moment, which is the major thing we're aiming for with this system. You've been a lot more thorough than I would have been though - I don't think I would have bothered with a full history like that to start with ;-)

Changed 6 years ago by openstreetmap@…

Patch to pull notes from uploaded gpx files (untested due to rmagick install/version issues plus gps_points has no id field apparently needed by trace.rb#186 exists?)

comment:2 Changed 4 years ago by tom@…

  • Priority changed from major to minor
  • Milestone set to Wishlist

comment:3 Changed 3 years ago by TomH

  • Owner changed from tom@… to rails-dev@…

comment:4 Changed 10 months ago by Andy Allan

  • Status changed from new to closed
  • Resolution set to fixed

Notes are now live on the website, and available via the API.

Note: See TracTickets for help on using tickets.