Make it easier to get page title from map resources #332

Closed
philipn opened this Issue Sep 1, 2012 · 2 comments

Comments

Projects
None yet
2 participants
Owner

philipn commented Sep 1, 2012

>> We are showing a series of GPS locations and the content title name when the
>> user clicks a map marker. I think this is a really common use case. You
>> could also imagine a non-map scenario showing a list of pages by title.
>>
>> Currently to get this title, we need to query the page: api/page/Some_Page
>> for every result in the list. So for example, to get a list of the nearest
>> 25 pages and their titles, we need to make 26 consecutive queries (one to
>> api/map, and 25 to api/Page/FooN). This is problematic for performance
>> reasons, both in app responsiveness, and on server load for the wiki website
>> serving the content
>>
>> Proposal: Include an additional property in the "geom" object: page_title.
>> It is just the page title.

@groovecoder groovecoder added a commit to groovecoder/localwiki that referenced this issue Mar 26, 2013

@groovecoder groovecoder fixes #332 - include full page data in maps api 15a7670
Owner

philipn commented Mar 26, 2013

I'm not sure about this. It makes loading the map via the API potentially much slower on such devices. I think maybe we want a query string, say, full=True, that turns this on.

Member

groovecoder commented Mar 26, 2013

Yeah that's a much better idea. I was trying to figure out how to do this with tastypie hydration but a url param is much better and easier.

@groovecoder groovecoder added a commit to groovecoder/localwiki that referenced this issue Mar 26, 2013

@groovecoder groovecoder fixes #332 - include full page data in maps api cf592a1

@groovecoder groovecoder added a commit to groovecoder/localwiki that referenced this issue Mar 26, 2013

@groovecoder groovecoder fixes #332 - include full page data in maps api a0071e4

philipn closed this in b87b02a Mar 26, 2013

@philipn philipn added a commit that referenced this issue Mar 26, 2013

@philipn philipn Merge pull request #459 from groovecoder/page-title-in-map-api-332
fixes #332 - include full page data in maps api
79617d7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment