fix: docker build requires smtp host (#672)

set a default for smtp host and add an action
for testing docker builds on each pull request

.github/workflows/ci.yml

@@ -14,8 +14,8 @@ env:
   HUSKY: 0
 
 jobs:
-  build:
+  build_app:
-    name: Build
+    name: Build App
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -37,3 +37,16 @@ jobs:
 
       - name: Build
         run: npm run build
+
+  build_docker:
+    name: Build Docker Image
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+        with:
+          fetch-depth: 2
+
+      - name: Build Docker Image
+        run: ./docker/build.sh
+

apps/marketing/package.json

@@ -13,6 +13,7 @@
     "copy:pdfjs": "node ../../scripts/copy-pdfjs.cjs"
   },
   "dependencies": {
+    "@documenso/assets": "*",
     "@documenso/lib": "*",
     "@documenso/tailwind-config": "*",
     "@documenso/trpc": "*",

apps/web/package.json

@@ -13,6 +13,7 @@
     "copy:pdfjs": "node ../../scripts/copy-pdfjs.cjs"
   },
   "dependencies": {
+    "@documenso/assets": "*",
     "@documenso/ee": "*",
     "@documenso/lib": "*",
     "@documenso/prisma": "*",

apps/web/src/components/(dashboard)/layout/desktop-nav.tsx

@@ -16,7 +16,9 @@ export const DesktopNav = ({ className, ...props }: DesktopNavProps) => {
   // const pathname = usePathname();
   const [open, setOpen] = useState(false);
 
-  const isMacOS = /Macintosh|Mac\s+OS\s+X/i.test(navigator?.userAgent || 'unknown');
+  const userAgent = typeof navigator !== 'undefined' ? navigator.userAgent : 'unknown';
+
+  const isMacOS = /Macintosh|Mac\s+OS\s+X/i.test(userAgent);
   const modifierKey = isMacOS ? '⌘' : 'Ctrl';
 
   return (

package-lock.json

@@ -36,6 +36,7 @@
       "version": "0.1.0",
       "license": "AGPL-3.0",
       "dependencies": {
+        "@documenso/assets": "*",
         "@documenso/lib": "*",
         "@documenso/tailwind-config": "*",
         "@documenso/trpc": "*",
@@ -79,6 +80,7 @@
       "version": "0.1.0",
       "license": "AGPL-3.0",
       "dependencies": {
+        "@documenso/assets": "*",
         "@documenso/ee": "*",
         "@documenso/lib": "*",
         "@documenso/prisma": "*",

packages/email/mailer.ts

@@ -42,12 +42,8 @@ const getTransport = () => {
     });
   }
 
-  if (!process.env.NEXT_PRIVATE_SMTP_HOST) {
-    throw new Error('SMTP transport requires NEXT_PRIVATE_SMTP_HOST');
-  }
-
   return createTransport({
-    host: process.env.NEXT_PRIVATE_SMTP_HOST,
+    host: process.env.NEXT_PRIVATE_SMTP_HOST ?? 'localhost:2500',
     port: Number(process.env.NEXT_PRIVATE_SMTP_PORT) || 587,
     secure: process.env.NEXT_PRIVATE_SMTP_SECURE === 'true',
     auth: {