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 a bbox validator to ensure correct order and valid wgs84 coordinates #95

Merged
merged 3 commits into from
Jul 3, 2021

Conversation

geospatial-jeff
Copy link
Collaborator

@geospatial-jeff geospatial-jeff commented Jul 3, 2021

Adds a bbox validator to ensure order is correct and coordinates are valid WGS84.

Closes #94

@codecov-commenter
Copy link

codecov-commenter commented Jul 3, 2021

Codecov Report

Merging #95 (5e46c09) into master (45e2735) will increase coverage by 0.12%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   96.06%   96.18%   +0.12%     
==========================================
  Files          20       20              
  Lines         457      472      +15     
==========================================
+ Hits          439      454      +15     
  Misses         18       18              
Flag Coverage Δ
unittests 96.18% <100.00%> (+0.12%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
stac_pydantic/api/search.py 96.15% <100.00%> (+0.91%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45e2735...5e46c09. Read the comment docs.

@geospatial-jeff geospatial-jeff merged commit 521ecd8 into master Jul 3, 2021
@geospatial-jeff geospatial-jeff deleted the issue/94/validate-bbox-order branch July 3, 2021 19:51
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.

Validate bbox order in api search
2 participants