Provide image URLs for map JSON results

[jywarren]

Our API currently shows:

https://mapknitter.org/map/region/Gulf-Coast.json?minlon=-98.8&minlat=23.6&maxlon=-79.1&maxlat=31.8

Which just converts map records to json directly, and results in them having a lot of not very useful content that's not readable; the body attribute especially.

We should remove body and instead include an imageUrl parameter, by appending it like we do here:

mapknitter/app/controllers/maps_controller.rb

Lines 146 to 157 in bf0b340

	# used by leaflet to fetch corner coords of each warpable
	def images
	map = Map.find params[:id]
	warpables = []
	map.warpables.each do |warpable|
	warpables << warpable
	warpables.last[:nodes] = warpable.nodes_array
	warpables.last.src = warpable.image.url
	warpables.last.srcmedium = warpable.image.url(:medium)
	end
	render :json => warpables
	end

This would let us display an image in the popup on the Leaflet map in Leaflet Environmental Layers: publiclab/leaflet-environmental-layers#112

[kaustubh-nair]

Hey I'd like to take this issue!
If i understand it correctly, you want to display only the image urls for all warpables in each map right?

[jywarren]

Yes! As an attribute alongside other map data. We can just use the url of the first returned image for the map and not provide the attribute if there aren't any!

…

On Wed, Feb 20, 2019, 5:46 AM Kaustubh Nair ***@***.*** wrote: Hey I'd like to take this issue! If i understand it correctly, you want to display only the image urls for all warpables in each map right? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#346 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ-UIl5QO83sQHYMRIJRBqA2Ldd6Lks5vPVF6gaJpZM4bEInW> .