-
Notifications
You must be signed in to change notification settings - Fork 30
add dynamic connections docs #1029
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
Conversation
Deploying ddn-docs with
|
Latest commit: |
922e975
|
Status: | ✅ Deploy successful! |
Preview URL: | https://87b6de88.v3-docs-eny.pages.dev |
Branch Preview URL: | https://add-dynamic-postgres.v3-docs-eny.pages.dev |
@BenoitRanque Thanks for your PR! I've assigned @robertjdominguez to review it. |
DX: Assertion Testing✅ DiffThe author's contribution in the PR clearly explains the new functionality provided by the ✅ IntegratedThe introduction of the |
Can we add a link to the dynamic connection page from the postgres index page |
Or should this live under plugin tutorials? I know this is specific to postgres only right now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving so as not to block; please see the note below.
docs/reference/connectors/postgresql/dynamic-connections-tutorial.mdx
Outdated
Show resolved
Hide resolved
Dynamic connections specifically is a postgres feature. We could add a tutorial for ndc level caching, which could live with the other plugins. But that would mean we would be suggesting two different plugins for caching. Are we happy to do that? They're technically different, do we have a preference for where we'd rather folks implement caching? IMHO, we have a caching plugin already, so let's not muddy the waters. |
Description 📝
Quick Links 🚀
Assertion Tests 🤖
Users should understand postgres dynamic connections configuration
Users should be able to understand and follow the tutorial