Page MenuHomePhabricator

Allow readable URLs for externaldata in geojson
Closed, ResolvedPublic

Description

Having raw URL in the ExternalData element of geojson is unreadable. I propose that we allow this structure (converted to URL by the backend geojson sanitizer):

{
  "type": "externalData",

  // this basically becomes the protocol.  All other values in this object are interpreted based on the service
  "service": "geoshape",

  // For "geoshape" service, query may be either one string, on a list of strings (will be concatenated)
  "query": [
    "SELECT ?id ?description ?head ?title (SAMPLE(?fill) as ?fill)  WHERE {",
    "  ?id wdt:P31 wd:Q35657 .",
    ...
  ]
}

Event Timeline

Change 311147 had a related patch set uploaded (by Yurik):
Allow readable URLs for externaldata in geojson

https://gerrit.wikimedia.org/r/311147

Change 311147 merged by jenkins-bot:
Allow readable queries for externaldata in geojson

https://gerrit.wikimedia.org/r/311147

Change 312406 had a related patch set uploaded (by MaxSem):
Allow readable queries for externaldata in geojson

https://gerrit.wikimedia.org/r/312406

Change 312406 abandoned by MaxSem:
Allow readable queries for externaldata in geojson

https://gerrit.wikimedia.org/r/312406