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

Configure database connections #22

Merged
merged 3 commits into from
May 10, 2023
Merged

Configure database connections #22

merged 3 commits into from
May 10, 2023

Conversation

gadomski
Copy link
Member

@gadomski gadomski commented May 9, 2023

Related Issue(s):

Description:

Ports to pgstac-specific PRs that got merged to stac-fastapi before the breakout.

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.

@gadomski gadomski requested a review from pjhartzell May 9, 2023 21:08
@gadomski gadomski self-assigned this May 9, 2023
@gadomski gadomski merged commit c592830 into main May 10, 2023
6 checks passed
@gadomski gadomski deleted the issues/19-db-connections branch May 10, 2023 14:45
@gadomski gadomski added this to the 2.4.6 milestone May 11, 2023
gadomski added a commit that referenced this pull request Jun 13, 2023
This makes #22 less breaking.
@gadomski gadomski mentioned this pull request Jun 13, 2023
4 tasks
gadomski added a commit that referenced this pull request Jun 16, 2023
This makes #22 less breaking.
gadomski added a commit that referenced this pull request Jun 20, 2023
* fix: pass request by name into methods

This makes #22 less breaking.

* chore: update changelog
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.

Backport custom DB connections
3 participants