-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add production database connection to payload #23
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
base: main
Are you sure you want to change the base?
Conversation
802a490 to
9c27786
Compare
|
Check how I managed to resolve this issue in boring stack. Here's PR code https://github.com/tonik/boring-stack/pull/35/files |
6d447de to
93f7ef4
Compare
| }, | ||
| })`; | ||
|
|
||
| data = data.replace( |
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.
this place can produce errors if the payload changes the name for postgresAdapter.
Have you considered using a codemod here and replacing it via AST?
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.
This is a great idea! I think we should introduce codemods globally across the whole app when it comes to reading and writing files. This, however, will be done in a separate PR.
330c96d to
d40ce80
Compare
This PR introduces dynamic environments for Payload CMS, Vercel and local.