Skip to content

docs: rewrite database integration guide around modern drivers and best practices - #2477

Merged
bjohansebas merged 6 commits into
mainfrom
guide/database-integration-rewrite
Oct 24, 2026
Merged

bjohansebas merged 6 commits into
mainfrom
guide/database-integration-rewrite

Conversation

@bjohansebas

Copy link
Copy Markdown
Member

Replace the outdated per-database driver catalog (deprecated packages, callback-era APIs) with a single modern example using node:sqlite in cjs, mjs, and ts variants, links to the official docs of other popular drivers, an overview of ORMs and query builders, and a best practices section covering connection pooling, parameterized queries, credentials, error handling, request cancellation with req.signal, and graceful shutdown.

@bjohansebas
bjohansebas requested a review from a team as a code owner August 2, 2026 19:32
@netlify

netlify Bot commented Aug 2, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit b40f8e7
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/6adcff7dc9aba1839a6a76a6
😎 Deploy Preview https://deploy-preview-2477--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 98 (🟢 up 1 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@crandmck crandmck left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small wording suggestion. Also, it looks like link checking CI is failing.

Comment thread src/content/pages/en/guide/database-integration.mdx Outdated
@crandmck

crandmck commented Sep 7, 2026

Copy link
Copy Markdown
Member

@bjohansebas Do you still want to pursue this?

@bjohansebas
bjohansebas requested a review from crandmck October 2, 2026 16:31
bjohansebas and others added 5 commits October 24, 2026 13:49
…st practices

Replace the outdated per-database driver catalog (deprecated packages,
callback-era APIs) with a single modern example using node:sqlite in
cjs, mjs, and ts variants, links to the official docs of other popular
drivers, an overview of ORMs and query builders, and a best practices
section covering connection pooling, parameterized queries, credentials,
error handling, request cancellation with req.signal, and graceful
shutdown.
Co-authored-by: Rand McKinney <crandmck@yahoo.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@bjohansebas
bjohansebas force-pushed the guide/database-integration-rewrite branch from ec51582 to 1226b67 Compare October 24, 2026 18:49
@bjohansebas
bjohansebas merged commit 573dfed into main Oct 24, 2026
14 checks passed
@bjohansebas
bjohansebas deleted the guide/database-integration-rewrite branch October 24, 2026 19:01
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