Commit Graph

8 Commits

Author SHA1 Message Date
kjh2064 e5981769b9 fix: per-page WASM render mode, Contact checkbox binding, Telegram inquiry channel
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m11s
- Admin: replace the global @rendermode on <Routes>/<Router> with per-page
  render mode. Login.razor now prerenders (form visible before WASM loads);
  every other [Authorize] page stays prerender: false to avoid the
  AuthorizeRouteView blank-render regression from earlier attempts. Adds a
  "준비 중" -> "로그인" splash tied to WASM boot completion, and lets the
  authenticated-shell loading overlay stay up until AdminShell actually renders.
- Contact.cshtml: fix the "Agree" checkbox missing value="true" - a checked
  box sent the browser-default "on", which bool model binding can't parse,
  so ModelState.IsValid silently went false and OnPostAsync returned a blank
  form with no visible error on every submission. Validation summary widened
  from ModelOnly to All so this class of failure isn't silent again.
- TelegramInquiryNotificationService: read Telegram:InquiryChatId (falling
  back to ChatId) instead of only ChatId, matching the channel routing
  CLAUDE.md documents and deploy.yml already provisions as separate secrets.
- Reconcile CLAUDE.md's self-contradicting Phase 8 prerender notes (Phase 9),
  rewrite validate_admin_render.sh for the per-page design, and add a
  SmartAdmin 5.5 design reference section to DOUZONE_UX_GUIDE.md for future
  admin screens (existing screens unchanged, tracked as WBS P4-03).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-07-03 10:15:27 +09:00
kjh2064 d015bb6c92 fix: update validation script to accept both WebAssembly rendermode formats
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m32s
ISSUE:
Validation script required exact text 'InteractiveWebAssemblyRenderMode'
but Login.razor uses shortened form '@rendermode InteractiveWebAssembly'

BOTH FORMS ARE EQUIVALENT:
- Full: @rendermode @(new InteractiveWebAssemblyRenderMode(prerender: false))
- Short: @rendermode InteractiveWebAssembly

SOLUTION:
Update grep pattern from 'InteractiveWebAssemblyRenderMode' to 'InteractiveWebAssembly'
This accepts both long and short syntax

VALIDATION:
 App.razor: InteractiveWebAssemblyRenderMode(prerender: false)
 Login.razor: @rendermode InteractiveWebAssembly
 All 28+ pages: @rendermode InteractiveWebAssembly
 Architecture: Blazor WebAssembly CSR (client-side rendering)

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 03:13:16 +09:00
kjh2064 9b7e6eda4c refactor: update validation script to reflect prerender: false design
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m29s
CORE ISSUE RESOLVED:
prerender: true creates contradiction between SSR and CSR rendering modes,
causing infinite loop of blank screens and auth state conflicts.

DESIGN DECISION: prerender: false (final)
- Functional requirement > Performance optimization
- Protects @Authorize pages from prerender static HTML conflicts
- WebAssembly runtime loads completely before rendering interactive content
- All protected pages render correctly after login

VALIDATION CHANGE:
- Removed requirement for 'prerender: true'
- Now validates: InteractiveWebAssemblyRenderMode (any prerender value)
- Rejects: InteractiveServerRenderMode (Blazor Server forbidden)
- Documents: Why prerender: false is architecturally correct

Root cause documented in CLAUDE.md Phase 8.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 02:51:26 +09:00
kjh2064 484ece7a92 fix: update validation script for WebAssembly migration
TaxBaik CI/CD / build-and-deploy (push) Failing after 2m59s
Admin render harness now checks TaxBaik.Web.Client paths after Phase 8 migration.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 01:07:34 +09:00
kjh2064 d246071835 fix: restore Blazor WebAssembly render mode for ERP scalability
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m15s
Restore long-term architectural goal: Blazor WebAssembly for admin UI.

Rationale:
- TaxBaik is a semi-project for future ERP implementation
- ERP requires client-side scalability (no server-side state per user)
- WebAssembly offloads interactivity to browser (Circuit-free)
- Aligns with API-First + stateless backend design

Changes:
- App.razor: InteractiveWebAssemblyRenderMode (prerender: true)
- Routes: InteractiveWebAssemblyRenderMode (prerender: true)
- Login.razor: InteractiveWebAssemblyRenderMode (prerender: true)
- Program.cs: AddInteractiveWebAssemblyComponents()
- Updated validation script to enforce WebAssembly mode

Tradeoff accepted: Blazor WebAssembly bootstrap time vs future ERP extensibility.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-03 00:05:39 +09:00
kjh2064 ba981e7332 fix: resolve admin interactivity by unifying to Server render mode
TaxBaik CI/CD / build-and-deploy (push) Successful in 2m17s
Problem: Mixed WebAssembly (App) and Server (Login) render modes caused interaction breakage after login. Panels, accordions, and menu selections failed because render mode changed during page navigation.

Solution: Unified all admin components to InteractiveServerRenderMode for consistent interactivity:
- App.razor: Routes and HeadOutlet use InteractiveServerRenderMode
- Login.razor: Already uses InteractiveServerRenderMode
- Program.cs: Removed WebAssembly component registration

Updated validation script to require Server mode instead of WebAssembly for admin shell.

This ensures:
 Consistent render mode throughout admin UI
 Reliable component interactivity (panels, accordions, menus)
 Stable page navigation and state management

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
2026-07-02 23:58:45 +09:00
kjh2064 b1ac7129d9 Harden common code and render harness policies 2026-07-02 17:02:02 +09:00
kjh2064 e6253fdc83 chore(ci): guard admin webassembly render mode 2026-07-02 14:52:29 +09:00