Conversation
|
Why did you close it? The changes you made was fine and better. |
|
I made the PR without testing. It failed when deploying on Vercel (404 error), so more tweaking needed probably. But yes, the code to load environment variables is definitely worth merging. You may want to change your DB details also, seeing as they've been committed. |
| @@ -0,0 +1,29 @@ | |||
| # versel.json | |||
There was a problem hiding this comment.
Should the file name be versel.json or vercel.json? Also technically a comment in JSON does not work.
|
@chalkie11 Are you deploying this on Vercel? Good luck for that! And Yeah DB details should be updated. I think I remember it now, a developer changed the DB configurations with their own conf. some time ago. Good from my side as you updated it for good. If you need any assistance or stuck into some problem while deplying this project then please let me know. @lahwaacz You're right, it should be |
No description provided.