-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: add production database connection to payload #23
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.