-
Notifications
You must be signed in to change notification settings - Fork 2
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
Syncronize dev-branch #728
Conversation
* Remove watchdog from Dockerfile * Bump node to 18 😎 * Change backend name from kaaf to server * Split dependencies into base and prod * Change endpoint from /kaaf to /generate * Wrap mail code in if(prod), and use consolemail * Only import sentry and mail in prod
And this works ey? If I remember correctly I did a bunch of random stuff in dev once 🤔 And not everything worked 🤔 |
Should probably test some more - but I have sent myself a couple of mails through it and so far it seems to work as expected. |
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.
Haha you’re an MVP!! NextUI let’s go (at least until we replace it with Lego-bricks next week😁😁)
Ah fuck ._. |
As far as I can see this stuff has never been through a PR so I figured it was time to get it out there.
Just finished up the rebasing, so I haven't looked through this PR very thoroughly, although everything seems to run fine in the dev environment.
Added a last commit to fixup some of the things I got wrong - as it didn't seem worth it to stop at every one of the 290 commits it was behind to ensure it ran perfectly.