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.

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

add support for OGC API - Features - Part 3: CQL #798

Merged
merged 1 commit into from Oct 21, 2021
Merged

add support for OGC API - Features - Part 3: CQL #798

merged 1 commit into from Oct 21, 2021

Conversation

tomkralidis
Copy link
Member

Adds support for https://docs.ogc.org/DRAFTS/19-079.html

Note that tests/test_ogcapi_records_pycsw.py will pass once geopython/pycsw#715 goes live on https://demo.pycsw.org/cite.

@tomkralidis tomkralidis reopened this Oct 20, 2021
@tomkralidis
Copy link
Member Author

FYI test failures are unrelated/fixed in #797

@cehbrecht
Copy link
Contributor

FYI test failures are unrelated/fixed in #797

merged #797.

@cehbrecht
Copy link
Contributor

Note that tests/test_ogcapi_records_pycsw.py will pass once geopython/pycsw#715 goes live on https://demo.pycsw.org/cite.

We can switch to use xfail for online tests. Online tests would be optional to pass. But we then need also offline tests ... using mock or so ? A task for the next PRs ...

@coveralls
Copy link

coveralls commented Oct 21, 2021 •
edited

Coverage Status

Coverage increased (+0.4%) to 59.674% when pulling 93fe90a on tomkralidis:ogcapi-cql into 9ac1444 on geopython:master.

@tomkralidis tomkralidis merged commit 4a73688 into geopython:master Oct 21, 2021
@tomkralidis tomkralidis deleted the ogcapi-cql branch October 21, 2021 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalxas kalxas kalxas approved these changes

@cehbrecht cehbrecht cehbrecht approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@tomkralidis @cehbrecht @coveralls @kalxas