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

update to sfapi v3.0.0 #282

Merged
merged 14 commits into from
Aug 6, 2024
Merged

update to sfapi v3.0.0 #282

merged 14 commits into from
Aug 6, 2024

Conversation

jonhealy1
Copy link
Collaborator

@jonhealy1 jonhealy1 commented Aug 3, 2024

Related Issue(s):

Description:

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable
  • Changes are added to the changelog

@jonhealy1
Copy link
Collaborator Author

There are 2 tests that I skipped related to pagination. They should be fixed before this is merged

@jonhealy1 jonhealy1 marked this pull request as ready for review August 3, 2024 23:55
Copy link
Collaborator

@jamesfisher-geo jamesfisher-geo left a comment

Choose a reason for hiding this comment

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

Looks great

@jamesfisher-geo
Copy link
Collaborator

@jonhealy1 removed the code discussed in #283.

@jonhealy1
Copy link
Collaborator Author

@jamesfisher-gis thank you

@jonhealy1 jonhealy1 merged commit 6fb8278 into main Aug 6, 2024
15 checks passed
@jonhealy1 jonhealy1 deleted the stac-fastapi-v3.0.0 branch August 6, 2024 18:05
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