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

Gs 2422 pgstac bug fix #6

Merged
merged 10 commits into from
May 9, 2023
Merged

Gs 2422 pgstac bug fix #6

merged 10 commits into from
May 9, 2023

Conversation

ConorWalsh18
Copy link

Related Issue(s):

Description:

PR Checklist:

  • pre-commit hooks pass locally
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

keul and others added 10 commits March 20, 2023 17:23
* Update reference to images (latest is now main)

* Update reference to images (latest is now main)
* Avoid fstring in queries

* Reorder import

* chore: update changelog

---------

Co-authored-by: Pete Gadomski <pete.gadomski@gmail.com>
* deps: remove ceil on fastapi version

It was added for some reason, but the reason appears to be lost in time.

* test: fix for new TestClient

Described here: https://github.com/tiangolo/fastapi/releases/tag/0.87.0
…ls#538)

* Add functionality to customize db connection retrieval

* Add test for customizing the connection_getter

* Cleanup

* isort fixes

* flake8 fix

* Update typing

---------

Co-authored-by: David Bitner <bitner@dbspatial.com>
@ConorWalsh18 ConorWalsh18 requested a review from ishiland May 5, 2023 19:56
Copy link

@ishiland ishiland left a comment

Choose a reason for hiding this comment

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

rebuilt container and deployed to SIT, looks to be working correctly with latest migration (0.7.6)

@ConorWalsh18 ConorWalsh18 merged commit e37a7f4 into develop May 9, 2023
@ConorWalsh18 ConorWalsh18 deleted the GS-2422-pgstac-bug-fix branch May 9, 2023 19:58
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.

6 participants