Skip to content

Links property for response at path / is optional? #69

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.

Sign up for GitHub

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

Jump to bottom
Closed
ahmedOpeyemi opened this issue Mar 7, 2018 · 2 comments
Closed

Links property for response at path / is optional? #69

ahmedOpeyemi opened this issue Mar 7, 2018 · 2 comments
Assignees
@cportele
Labels
bug Part 1: Core Issue related to Part 1 - Core progress: pull request available
Milestone
Part 1, First Dra...

Comments

@ahmedOpeyemi
Copy link

The schema defined on the spec document as response to a GET request on the / path shows the property links to be optional. However in the reference yaml document that contains the schema, it is marked required.
@cportele cportele self-assigned this Mar 8, 2018
@cportele cportele added this to the Part 1, First Draft Release milestone Mar 8, 2018
@cportele
Copy link
Member

cportele commented Mar 8, 2018

Thanks, @ahmedOpeyemi. Since /req/core/fc-md-links requires that links are provided, the document was wrong. This has been fixed in pull request #70.

@cportele cportele mentioned this issue Mar 8, 2018
Merged
cportele added a commit that referenced this issue Mar 12, 2018
@cportele
Merge pull request #70 from opengeospatial/wfs3hackathon
e4146e7 
WFS 3.0 Hackathon (issues #59, #61, #62, #63, #64, #69, #72, #77, #78)
@cportele
Copy link
Member

resolved as part of pull request #70

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cportele cportele

Labels
bug Part 1: Core Issue related to Part 1 - Core progress: pull request available
Projects
Features Part 1: Core
Status: Done
Milestone
Part 1, First Draft Release
Development

No branches or pull requests
2 participants
@cportele @ahmedOpeyemi