fix: handle plain month number in attendance route, not just YYYY-MM

AttendanceAdmin sends ?year=YYYY&month=M (separate params), but the route handler assumed month was always in "YYYY-MM" format. When query.month was just "4", the split produced NaN for month and 4 for year, causing the service to skip the month filter entirely. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
v1.6.0: fix offer items mobile layout and localStorage draft save/restore
2026-04-29 16:50:20 +02:00 · 2026-04-29 15:23:42 +02:00 · 2026-04-29 09:29:47 +02:00 · 2026-04-29 09:08:15 +02:00 · 2026-04-29 08:29:10 +02:00 · 2026-04-29 08:04:22 +02:00
7 changed files with 261 additions and 118 deletions
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "app-ts",
-  "version": "1.5.6",
+  "version": "1.6.1",
  "description": "",
  "main": "dist/server.js",
  "scripts": {
--- a/src/admin/offers.css
+++ b/src/admin/offers.css
@@ -491,10 +491,69 @@
  display: none;
 }

+@media (max-width: 768px) {
+  .offers-items-table .admin-table td .admin-form-input {
+    font-size: 16px;
+    min-height: 44px;
+    padding: 9px 10px;
+  }
+
+  /* Give the description column more room by shrinking numeric columns */
+  .offers-col-qty {
+    width: 4rem !important;
+  }
+  .offers-col-unit {
+    width: 4rem !important;
+  }
+  .offers-col-price {
+    width: 5.5rem !important;
+  }
+  .offers-col-included {
+    width: 3.5rem !important;
+  }
+  .offers-col-total {
+    width: 5.5rem !important;
+  }
+  .offers-col-del {
+    width: 2.5rem !important;
+  }
+}
+
@media (max-width: 640px) {
  .offers-items-table {
    margin: 0 -1rem;
    width: calc(100% + 2rem);
+    border-radius: 0;
+    border-left: none;
+    border-right: none;
+  }
+
+  .offers-items-table .admin-table {
+    min-width: 700px;
+  }
+
+  .offers-items-table .admin-table td {
+    padding: 6px;
+    font-size: 12px;
+  }
+
+  .offers-items-table .admin-table th {
+    font-size: 10px;
+    padding: 6px;
+  }
+
+  /* Further reduce numeric columns on very small screens */
+  .offers-col-qty {
+    width: 3.5rem !important;
+  }
+  .offers-col-unit {
+    width: 3.5rem !important;
+  }
+  .offers-col-price {
+    width: 5rem !important;
+  }
+  .offers-col-total {
+    width: 5rem !important;
  }
 }

--- a/src/admin/pages/AuditLog.tsx
+++ b/src/admin/pages/AuditLog.tsx
@@ -391,6 +391,10 @@ export default function AuditLog() {
      >
        <div className="admin-card-body">
          <div className="admin-table-responsive">
+            <Skeleton
+              name="audit-log-rows"
+              loading={isPending}
+              fixture={
                <table className="admin-table">
                  <thead>
                    <tr>
@@ -403,20 +407,9 @@ export default function AuditLog() {
                    </tr>
                  </thead>
                  <tbody>
-                <Skeleton
-                  name="audit-log-rows"
-                  loading={isPending}
-                  fixture={
-                    <div style={{ padding: "1rem" }}>
                    {Array.from({ length: 10 }, (_, i) => (
-                        <div
-                          key={i}
-                          style={{
-                            display: "flex",
-                            gap: "1rem",
-                            marginBottom: "0.75rem",
-                          }}
-                        >
+                      <tr key={i}>
+                        <td>
                          <div
                            style={{
                              width: 110,
@@ -425,6 +418,8 @@ export default function AuditLog() {
                              borderRadius: 4,
                            }}
                          />
+                        </td>
+                        <td>
                          <div
                            style={{
                              width: 80,
@@ -433,6 +428,8 @@ export default function AuditLog() {
                              borderRadius: 4,
                            }}
                          />
+                        </td>
+                        <td>
                          <div
                            style={{
                              width: 70,
@@ -441,6 +438,8 @@ export default function AuditLog() {
                              borderRadius: 10,
                            }}
                          />
+                        </td>
+                        <td>
                          <div
                            style={{
                              width: 80,
@@ -449,14 +448,18 @@ export default function AuditLog() {
                              borderRadius: 4,
                            }}
                          />
+                        </td>
+                        <td>
                          <div
                            style={{
-                              flex: 1,
+                              width: "100%",
                              height: 14,
                              background: "var(--bg-tertiary)",
                              borderRadius: 4,
                            }}
                          />
+                        </td>
+                        <td>
                          <div
                            style={{
                              width: 90,
@@ -465,12 +468,25 @@ export default function AuditLog() {
                              borderRadius: 4,
                            }}
                          />
-                        </div>
+                        </td>
+                      </tr>
                    ))}
-                    </div>
+                  </tbody>
+                </table>
              }
            >
-                  <>
+              <table className="admin-table">
+                <thead>
+                  <tr>
+                    <th>Čas</th>
+                    <th>Uživatel</th>
+                    <th>Akce</th>
+                    <th>Typ entity</th>
+                    <th>Popis</th>
+                    <th>IP</th>
+                  </tr>
+                </thead>
+                <tbody>
                  {logs.length === 0 && (
                    <tr>
                      <td colSpan={6}>
@@ -518,10 +534,9 @@ export default function AuditLog() {
                        <td className="admin-mono">{log.user_ip || "-"}</td>
                      </tr>
                    ))}
-                  </>
-                </Skeleton>
                </tbody>
              </table>
+            </Skeleton>
          </div>

          <Pagination
--- a/src/admin/pages/OfferDetail.tsx
+++ b/src/admin/pages/OfferDetail.tsx
@@ -274,9 +274,10 @@ function loadOfferDraft(): {
  sections?: unknown[];
 } | null {
  try {
-    const raw = localStorage.getItem("boha_offer_draft");
+    const raw = localStorage.getItem(DRAFT_KEY);
    return raw ? JSON.parse(raw) : null;
-  } catch {
+  } catch (e) {
+    console.error("Failed to load offer draft:", e);
    return null;
  }
 }
@@ -353,17 +354,28 @@ export default function OfferDetail() {
    const draft = loadOfferDraft();
    if (draft?.form) {
      return {
-        ...emptyForm,
+        quotation_number:
+          (draft.form.quotation_number as string) || emptyForm.quotation_number,
        project_code:
          (draft.form.project_code as string) || emptyForm.project_code,
+        customer_id:
+          (draft.form.customer_id as number | null) ?? emptyForm.customer_id,
        customer_name:
          (draft.form.customer_name as string) || emptyForm.customer_name,
        created_at: (draft.form.created_at as string) || emptyForm.created_at,
        valid_until:
          (draft.form.valid_until as string) || emptyForm.valid_until,
        currency: (draft.form.currency as string) || emptyForm.currency,
-        customer_id:
-          (draft.form.customer_id as number | null) ?? emptyForm.customer_id,
+        language: (draft.form.language as string) || emptyForm.language,
+        vat_rate: (draft.form.vat_rate as number) ?? emptyForm.vat_rate,
+        apply_vat: (draft.form.apply_vat as boolean) ?? emptyForm.apply_vat,
+        exchange_rate:
+          (draft.form.exchange_rate as string) || emptyForm.exchange_rate,
+        scope_title:
+          (draft.form.scope_title as string) || emptyForm.scope_title,
+        scope_description:
+          (draft.form.scope_description as string) ||
+          emptyForm.scope_description,
      };
    }
    return emptyForm;
@@ -590,22 +602,27 @@ export default function OfferDetail() {
  useEffect(() => {
    if (isEdit) return;
    try {
+      const data = JSON.parse(debouncedDraft);
      const draft = {
        form: {
-          project_code: form.project_code,
-          customer_id: form.customer_id,
-          customer_name: form.customer_name,
-          created_at: form.created_at,
-          valid_until: form.valid_until,
-          currency: form.currency,
+          project_code: data.form.project_code ?? "",
+          customer_id: data.form.customer_id ?? null,
+          customer_name: data.form.customer_name ?? "",
+          created_at: data.form.created_at ?? "",
+          valid_until: data.form.valid_until ?? "",
+          currency: data.form.currency ?? "CZK",
+          language: data.form.language ?? "EN",
+          vat_rate: data.form.vat_rate ?? 21,
+          apply_vat: data.form.apply_vat ?? false,
+          exchange_rate: data.form.exchange_rate ?? "",
        },
-        items,
-        sections,
+        items: data.items,
+        sections: data.sections,
        savedAt: new Date().toISOString(),
      };
      localStorage.setItem(DRAFT_KEY, JSON.stringify(draft));
-    } catch {
-      /* localStorage full or unavailable */
+    } catch (e) {
+      console.error("Failed to save offer draft:", e);
    }
  }, [debouncedDraft]); // eslint-disable-line react-hooks/exhaustive-deps

@@ -697,8 +714,8 @@ export default function OfferDetail() {
        if (!isEdit) {
          try {
            localStorage.removeItem(DRAFT_KEY);
-          } catch {
-            /* ignore */
+          } catch (e) {
+            console.error("Failed to remove offer draft:", e);
          }
        }
        if (!isEdit && result.data?.id) {
@@ -1283,7 +1300,7 @@ export default function OfferDetail() {
              </p>
            )}

-            <div className="admin-table-responsive">
+            <div className="offers-items-table">
              <DndContext
                sensors={dndSensors}
                collisionDetection={closestCenter}
@@ -1316,18 +1333,42 @@ export default function OfferDetail() {
                        <th style={{ width: "2.5rem", textAlign: "center" }}>
                          #
                        </th>
-                        <th>Popis</th>
-                        <th style={{ width: "5rem" }}>Množství</th>
-                        <th style={{ width: "5rem" }}>Jednotka</th>
-                        <th style={{ width: "7rem" }}>Cena/ks</th>
-                        <th style={{ width: "4rem", textAlign: "center" }}>
+                        <th className="offers-col-desc">Popis</th>
+                        <th
+                          className="offers-col-qty"
+                          style={{ width: "5rem" }}
+                        >
+                          Množství
+                        </th>
+                        <th
+                          className="offers-col-unit"
+                          style={{ width: "5rem" }}
+                        >
+                          Jednotka
+                        </th>
+                        <th
+                          className="offers-col-price"
+                          style={{ width: "7rem" }}
+                        >
+                          Cena/ks
+                        </th>
+                        <th
+                          className="offers-col-included"
+                          style={{ width: "4rem", textAlign: "center" }}
+                        >
                          V ceně
                        </th>
-                        <th style={{ width: "7rem", textAlign: "right" }}>
+                        <th
+                          className="offers-col-total"
+                          style={{ width: "7rem", textAlign: "right" }}
+                        >
                          Celkem
                        </th>
                        {!isInvalidated && !isLockedByOther && (
-                          <th style={{ width: "3rem" }} />
+                          <th
+                            className="offers-col-del"
+                            style={{ width: "3rem" }}
+                          />
                        )}
                      </tr>
                    </thead>
--- a/src/admin/pages/OffersTemplates.tsx
+++ b/src/admin/pages/OffersTemplates.tsx
@@ -71,7 +71,7 @@ export default function OffersTemplates() {
        </div>
      </motion.div>

-      <div className="admin-tabs">
+      <div className="admin-tabs mb-4">
        <button
          className={`admin-tab ${activeTab === "items" ? "active" : ""}`}
          onClick={() => setActiveTab("items")}
--- a/src/admin/pages/Settings.tsx
+++ b/src/admin/pages/Settings.tsx
@@ -1546,15 +1546,31 @@ export default function Settings() {
                      </tbody>
                    </table>
                  ) : (
-                    <Skeleton
-                      name="settings-permissions"
-                      loading={
-                        !role.permissions || role.permissions.length === 0
-                      }
-                      fixture={<span>...</span>}
+                    <div
+                      style={{
+                        padding: "1rem",
+                        display: "flex",
+                        flexDirection: "column",
+                        gap: "0.5rem",
+                      }}
                    >
-                      <span>{role.permissions?.length || 0} oprávnění</span>
-                    </Skeleton>
+                      <div
+                        style={{
+                          height: "0.75rem",
+                          background: "#e0e0e0",
+                          borderRadius: "4px",
+                          width: "60%",
+                        }}
+                      />
+                      <div
+                        style={{
+                          height: "0.75rem",
+                          background: "#e0e0e0",
+                          borderRadius: "4px",
+                          width: "40%",
+                        }}
+                      />
+                    </div>
                  )}
                </div>
              </motion.div>
--- a/src/routes/admin/attendance.ts
+++ b/src/routes/admin/attendance.ts
@@ -220,8 +220,20 @@ export default async function attendanceRoutes(
      userId,
      isAdmin,
      authUserId: authData.userId,
-      month: query.month ? Number(query.month) : undefined,
-      year: query.year ? Number(query.year) : undefined,
+      month: query.month
+        ? String(query.month).includes("-")
+          ? Number(String(query.month).split("-")[1])
+          : Number(query.month)
+        : undefined,
+      year: query.month
+        ? String(query.month).includes("-")
+          ? Number(String(query.month).split("-")[0])
+          : query.year
+            ? Number(query.year)
+            : undefined
+        : query.year
+          ? Number(query.year)
+          : undefined,
    });

    return reply.send({
Author	SHA1	Message	Date
BOHA	e4f14a24b7	fix: handle plain month number in attendance route, not just YYYY-MM AttendanceAdmin sends ?year=YYYY&month=M (separate params), but the route handler assumed month was always in "YYYY-MM" format. When query.month was just "4", the split produced NaN for month and 4 for year, causing the service to skip the month filter entirely. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 16:50:20 +02:00
BOHA	3bd0d055d9	v1.6.0: fix offer items mobile layout and localStorage draft save/restore - Fix items table description column width on mobile (was ~82px, now ~260px) - Activate unused offers-items-table CSS class in OfferDetail form - Save all form fields to localStorage draft (language, VAT, exchange rate were missing) - Use DRAFT_KEY constant in loadOfferDraft, add error logging Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 15:23:42 +02:00
BOHA	746d17e182	fix: parse YYYY-MM month filter correctly in attendance history The frontend sends month as "YYYY-MM" but the route handler was passing it through Number() which parsed only the year portion, causing the service to ignore the month filter entirely. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 09:29:47 +02:00
BOHA	e96e51598a	v1.5.8: fix audit log table layout (Skeleton outside tbody) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 09:08:15 +02:00
BOHA	9abec36f07	v1.5.7: fix Settings system tab crash and OffersTemplates tab gap Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 08:29:10 +02:00
BOHA	ecd8e3679f	fix: replace stray role reference in system settings tab with inline placeholder Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>	2026-04-29 08:04:22 +02:00