Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(configuration): Add pool and connection object to SQL database default configuration #3023

Merged
merged 1 commit into from
Jan 29, 2023
Merged

fix(configuration): Add pool and connection object to SQL database default configuration #3023

merged 1 commit into from
Jan 29, 2023

Conversation

daffl
Copy link
Member

@daffl daffl commented Jan 28, 2023

Closes #2784

@netlify
Copy link

netlify bot commented Jan 28, 2023 •
edited
Loading

✅ Deploy Preview for feathers-dove ready!

Name Link
🔨 Latest commit 72aa204
🔍 Latest deploy log https://app.netlify.com/sites/feathers-dove/deploys/63d5a52190fb8b00089ad38a
😎 Deploy Preview https://deploy-preview-3023--feathers-dove.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

marshallswain
marshallswain approved these changes Jan 29, 2023
View reviewed changes
Copy link
Member

@marshallswain marshallswain left a comment

Choose a reason for hiding this comment

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

:shipit:

@daffl daffl merged commit 092c749 into dove Jan 29, 2023
@daffl daffl deleted the 2784-knex-connection-config branch January 29, 2023 17:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@marshallswain marshallswain marshallswain approved these changes

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

Successfully merging this pull request may close these issues.

Documentation for knex and connections
2 participants
@daffl @marshallswain