buda-base / lds-pdi

http://purl.bdrc.io BDRC Linked Data Server
Apache License 2.0
2 stars 0 forks source link

templates API #27

Closed eroux closed 6 years ago

eroux commented 6 years ago

A few things to fix on the /resource/templates/ API:

eroux commented 6 years ago

In fact, I think maybe using /query instead of /resource/template would make more sense, I don't think the /resource prefix is that relevant here... there is a little bit of API design to do here, creating a swagger file first will probably help

eroux commented 6 years ago

Also, http://buda1.bdrc.io/queries returns links with the .arq suffix, which don't work... but using some best practices like this one would make more sense anyways, the output would look something like:

[
  {
    "id": "PersonNames_byName",
    "href": "/queries/PersonNames_byName",
    "description": "Gets some basic info on persons with names matching the argument"
  },
...
]

(note that this would require a "description" field on the queries, which would be very useful)

xristy commented 6 years ago

+1 These are great suggestions.

MarcAgate commented 6 years ago

All points implemented as of commit 6961186 (Except for swagger file which requires some thinking and time)

eroux commented 6 years ago

looks good, thanks! the swagger file has its own issue, closing this one