Skip to content
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

Add getCollectionItemsUrl Method for OGC API #32

Merged
merged 3 commits into from
Apr 8, 2024

Conversation

ronitjadhav
Copy link
Member

This pull request introduces the getCollectionItemsUrl method in the OgcApiEndpoint class. This method is designed to generate a URL for the items of a specified collection, with optional query parameters.

The parameters include query, outputFormat, maxFeatures, skipGeometry, offset, outputCrs, extent, extentCrs, and properties. The method returns a promise that resolves to the URL as a string or rejects if an error occurs.

In addition to the method, this PR also includes unit tests to ensure the correct functionality of the getCollectionItemsUrl method.

This addition enhances the functionality of the OgcApiEndpoint class, providing more flexibility and options for users when working with collection items.

@ronitjadhav ronitjadhav requested a review from jahow April 5, 2024 09:57
Copy link
Member

@jahow jahow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think it looks good, there's a few parameters that are not supported in there though. In the future we might want to read what the endpoint advertises as supported (e.g. crs, formats...) to make sure the request is valid, but for now this is good :)

src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
src/ogc-api/endpoint.ts Outdated Show resolved Hide resolved
Copy link
Member

@jahow jahow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good, thanks a lot :)

@ronitjadhav ronitjadhav merged commit 09e0dd8 into main Apr 8, 2024
1 check passed
@ronitjadhav ronitjadhav deleted the ogcapi-collection-item-url branch April 8, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants