-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Publisher information in the API #508
Comments
It would be good to see I can see a case for including all fields/properties that are shown in the current dashboard. That is:
|
@devinbalkind @mrshll1001 @greggish @MikeThacker1 @Cskyleryoung This is the first issue created since we've had the committee in place, so keen to ensure if follows the process It has been 'Triaged' (we created it), 'Incubated' (not sure about this term though @devinbalkind) and we've labelled it 'minor' so is now at stage 3 'Prioritize' which I've created as a label and applied. Next is 'technical committee review' then 'proposal' which is where I think we'll need to work out how we 'Technical Stewards' and you 'The committee' work together. I've added the sections from the proposal doc to help us consider this. It could be worth us looking at each section and thinking about how/if the committee feeds in to each? Summary
Background and Motivation The problem we’re trying to solve
The benefits of the proposed update
The evidence for the need
Proposal Details
Schema changes
API Changes
[You should use additional subsections for your Proposal if needed] Further Considerations
Risks
Documentation
Considerations for existing Profiles
Considerations for existing Publishers
Considerations for Data Users
|
Do we think the committee would like to be more involved/hands with assessing issues at the level of the three sections below, or are seeing these as something we (technical stewards) lead on and run by the committee for comments/input? Background and Motivation
The benefits of the proposed update
The evidence for the need
|
Thanks for tagging me in here. I think we're still in the Issue Clarification period on this. During this period we should develop a good idea of what is being proposed without having a finalized, codified proposal. Mike suggested an approach and fields that he'd like to see included. Is that what is going to be proposed? The ODSC template you posted seems very thorough. If the people who want to address this issue want to use that template I think that'd be awesome. But I also think if you just wanted to post a concrete solution to the issue: a description of the approach and schema, maybe that'd be sufficient. |
On a recent call with implementors who were working on implementing the UK Profile, it came to light that we were all expecting that the API specification should include some fields for describing the publisher.
Other standards usually include this in their packaging schemas, making it part of the response or publication metadata.
This might not necessarily work for HSDS due to not having a packaging schema. So I think we have a few options if we want to address this in the short term:
publisher
object as part of the root (/
) API response.publisher
object to thePage
object used to wrap API responses to other endpointsThere could also be other options, and these are not mutually exclusive. We should also determine what format
publisher
should take. Should it be a few bare-minimum fields for brevity and being to-the-point, or should it be an instance ofOrganization
allowing for a rich description of the publisher using HSDS standard fields?The text was updated successfully, but these errors were encountered: