fix(254): move invoice modal buttons to sticky footer

fix(254): modal scrollable with sticky footer
Merge pull request 'fix: remove public COPY — no public dir in repo' (#24 ) from fix/remove-public-copy into main
2026-04-01 07:45:02 +00:00 · 2026-04-01 07:44:53 +00:00 · 2026-04-01 05:03:32 +00:00 · 2026-04-01 05:03:16 +00:00 · 2026-04-01 04:56:36 +00:00 · 2026-04-01 04:56:21 +00:00
12 changed files with 68 additions and 18 deletions
--- a/11
+++ b/11
@@ -1,6 +1,5 @@
 FROM node:22-alpine AS builder
 WORKDIR /app
-# Install ALL deps (including devDeps like tailwindcss, typescript)
 ENV NODE_ENV=development
 COPY package*.json ./
 RUN npm ci
@@ -12,9 +11,11 @@ FROM node:22-alpine AS runner
 WORKDIR /app
 ENV NODE_ENV=production
 ENV NEXT_TELEMETRY_DISABLED=1
-COPY --from=builder /app/.next/standalone ./
-COPY --from=builder /app/.next/static ./.next/static
-COPY --from=builder /app/public ./public
+
+COPY --from=builder /app/package*.json ./
+COPY --from=builder /app/.next ./.next
+COPY --from=builder /app/node_modules ./node_modules
+
 EXPOSE 3000
 ENV PORT=3000
-CMD ["node", "server.js"]
+CMD ["node_modules/.bin/next", "start"]
--- a/app/(app)/accounting/reports/page.tsx
+++ b/app/(app)/accounting/reports/page.tsx
@@ -1,4 +1,7 @@
 "use client";
+export const dynamic = "force-dynamic";
+
+

 import { useState } from "react";
 import { useQuery } from "@tanstack/react-query";
--- a/app/(app)/dashboard/page.tsx
+++ b/app/(app)/dashboard/page.tsx
@@ -1,4 +1,7 @@
 "use client";
+export const dynamic = "force-dynamic";
+
+

 import { useQuery, useMutation } from "@tanstack/react-query";
 import { useRouter } from "next/navigation";
--- a/app/(app)/invoices/page.tsx
+++ b/app/(app)/invoices/page.tsx
@@ -154,7 +154,20 @@ export default function InvoicesPage() {
      </Card>

      {/* Invoice Detail Modal */}
-      <Modal isOpen={!!selected} onClose={() => setSelected(null)} title={`Invoice ${selected?.invoiceNumber ?? ""}`} className="max-w-lg">
+      <Modal
+        isOpen={!!selected}
+        onClose={() => setSelected(null)}
+        title={`Invoice ${selected?.invoiceNumber ?? ""}`}
+        className="max-w-lg"
+        footer={selected ? (
+          <>
+            {selected.status !== "VOID" && selected.status !== "PAID" && (
+              <Button variant="danger" size="sm" onClick={() => voidInvoice.mutate(selected.id)} isLoading={voidInvoice.isPending}>Void Invoice</Button>
+            )}
+            <Button variant="outline" size="sm" onClick={() => setSelected(null)}>Close</Button>
+          </>
+        ) : undefined}
+      >
        {selected && (
          <div className="space-y-4">
            <div className="bg-gray-50 rounded-lg p-4 space-y-2 text-sm">
@@ -196,13 +209,6 @@ export default function InvoicesPage() {
                  disabled={!payForm.amount}>Record Payment</Button>
              </div>
            )}
-
-            <div className="flex justify-between pt-1">
-              {selected.status !== "VOID" && selected.status !== "PAID" && (
-                <Button variant="danger" size="sm" onClick={() => voidInvoice.mutate(selected.id)} isLoading={voidInvoice.isPending}>Void Invoice</Button>
-              )}
-              <Button variant="outline" size="sm" onClick={() => setSelected(null)} className="ml-auto">Close</Button>
-            </div>
          </div>
        )}
      </Modal>
--- a/app/(app)/layout.tsx
+++ b/app/(app)/layout.tsx
@@ -1,4 +1,5 @@
 'use client';
+export const dynamic = 'force-dynamic';

 import { useEffect, useState } from 'react';
 import { useRouter } from 'next/navigation';
--- a/app/(app)/reports/page.tsx
+++ b/app/(app)/reports/page.tsx
@@ -1,4 +1,7 @@
 "use client";
+export const dynamic = "force-dynamic";
+
+

 import { useState } from "react";
 import { useQuery } from "@tanstack/react-query";
--- a/app/layout.tsx
+++ b/app/layout.tsx
@@ -6,6 +6,8 @@ import Providers from '@/components/providers';
 // Fonts are loaded via CSS @import in globals.css (runtime CDN load).
 // CSS vars are set directly in globals.css :root — no JS injection needed.

+export const dynamic = 'force-dynamic';
+
 export const metadata: Metadata = {
  title: 'FiberOps Admin',
  description: 'ISP Management Platform',
--- a/next.config.mjs
+++ b/next.config.mjs
@@ -1,6 +1,5 @@
 /** @type {import('next').NextConfig} */
 const nextConfig = {
-  output: 'standalone',
  eslint: { ignoreDuringBuilds: true },
  typescript: { ignoreBuildErrors: true },
 };
--- a/pages/_document.tsx
+++ b/pages/_document.tsx
@@ -0,0 +1,12 @@
+import { Html, Head, Main, NextScript } from 'next/document';
+export default function Document() {
+  return (
+    <Html lang="en">
+      <Head />
+      <body>
+        <Main />
+        <NextScript />
+      </body>
+    </Html>
+  );
+}
--- a/pages/_error.tsx
+++ b/pages/_error.tsx
@@ -0,0 +1,14 @@
+// Custom error page — prevents Html import issue in Next.js pages router
+export default function Error({ statusCode }: { statusCode?: number }) {
+  return (
+    <div style={{ padding: '2rem', fontFamily: 'sans-serif' }}>
+      <h1>{statusCode || 'Error'}</h1>
+      <p>{statusCode === 404 ? 'Page not found' : 'An error occurred'}</p>
+    </div>
+  );
+}
+
+Error.getInitialProps = ({ res, err }: any) => {
+  const statusCode = res ? res.statusCode : err ? err.statusCode : 404;
+  return { statusCode };
+};
--- a/public/.gitkeep
+++ b/public/.gitkeep
--- a/src/components/ui/Modal.tsx
+++ b/src/components/ui/Modal.tsx
@@ -9,10 +9,11 @@ interface ModalProps {
  onClose: () => void;
  title: string;
  children: React.ReactNode;
+  footer?: React.ReactNode;
  className?: string;
 }

-export function Modal({ isOpen, onClose, title, children, className }: ModalProps) {
+export function Modal({ isOpen, onClose, title, children, footer, className }: ModalProps) {
  const overlayRef = useRef<HTMLDivElement>(null);

  useEffect(() => {
@@ -31,8 +32,8 @@ export function Modal({ isOpen, onClose, title, children, className }: ModalProp
      className="fixed inset-0 z-50 flex items-center justify-center bg-black/40 p-4"
      onClick={(e) => e.target === overlayRef.current && onClose()}
    >
-      <div className={cn("w-full max-w-lg rounded-xl bg-white shadow-xl", className)}>
-        <div className="flex items-center justify-between border-b border-gray-100 px-6 py-4">
+      <div className={cn("w-full max-w-lg rounded-xl bg-white shadow-xl flex flex-col max-h-[90vh]", className)}>
+        <div className="flex items-center justify-between border-b border-gray-100 px-6 py-4 flex-shrink-0">
          <h3 className="text-base font-semibold text-gray-900">{title}</h3>
          <button
            onClick={onClose}
@@ -41,7 +42,12 @@ export function Modal({ isOpen, onClose, title, children, className }: ModalProp
            <X className="h-5 w-5" />
          </button>
        </div>
-        <div className="px-6 py-4">{children}</div>
+        <div className="px-6 py-4 overflow-y-auto flex-1">{children}</div>
+        {footer && (
+          <div className="flex items-center justify-end gap-2 border-t border-gray-100 px-6 py-3 flex-shrink-0">
+            {footer}
+          </div>
+        )}
      </div>
    </div>
  );
Author	SHA1	Message	Date
kibin	c5826a34d0	fix(254): move invoice modal buttons to sticky footer	2026-04-01 07:45:02 +00:00
kibin	7de6d899f1	fix(254): modal scrollable with sticky footer	2026-04-01 07:44:53 +00:00
kibin	e582eb1693	Merge pull request 'fix: remove public COPY — no public dir in repo' (#24 ) from fix/remove-public-copy into main	2026-04-01 05:03:32 +00:00
Nemo (Claude Code)	41a90ad76e	fix: remove public COPY from Dockerfile, add public/.gitkeep	2026-04-01 05:03:16 +00:00
kibin	b880137084	Merge pull request 'fix: pages/_error and _document for build' (#23 ) from fix/pages-error-handler into main	2026-04-01 04:56:36 +00:00
Nemo (Claude Code)	fca3194801	fix: add pages/_error and _document to fix Html import in build	2026-04-01 04:56:21 +00:00
kibin	22c1df67c1	Merge pull request 'fix: force-dynamic on layouts — skip SSR static gen' (#22 ) from fix/force-dynamic-all-pages into main	2026-04-01 04:49:51 +00:00
Nemo (Claude Code)	ef6b6a3ad4	fix: force-dynamic on app layout to skip SSR static gen for all pages	2026-04-01 04:49:20 +00:00
kibin	8156c1f207	Merge pull request 'fix: force-dynamic on recharts pages (FIBEROPS-249)' (#21 ) from fix/force-dynamic-recharts-pages into main	2026-04-01 04:39:57 +00:00
Nemo (Claude Code)	8a31ca0199	fix: force-dynamic on recharts pages to prevent SSR static export failure	2026-04-01 04:39:54 +00:00
kibin	d58b6bfd0b	Merge pull request 'fix: remove standalone output — run next start in Docker' (#20 ) from fix/no-standalone-output into main	2026-04-01 04:18:47 +00:00
Nemo (Claude Code)	e8b91468a1	fix: remove standalone output, run next start in Docker	2026-04-01 04:18:31 +00:00
kibin	ac60822134	Merge pull request 'fix: .dockerignore to exclude .next cache' (#19 ) from fix/dockerignore-next-cache into main	2026-04-01 04:09:30 +00:00