kjh2064/taxbaik
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity
203 Commits 17 Branches 0 Tags
fbdbbc7a1fb54f84468dd5d15aa5d18cabfdbf4f
Commit Graph

4 Commits

Author SHA1 Message Date
kjh2064 160afb7c7e refactor: Phase 7-2 Complete - Full Inquiry page API-First migration
TaxBaik CI/CD / build-and-deploy (push) Successful in 49s
Details 
**Blockers Fixed:**

1. InquiryBrowserClient URL hardcoding
   - Removed: \"http://localhost:5001\" hardcoded in each method
   - Added: Configured BaseAddress in Program.cs
   - Now uses: Relative paths (\"inquiry\", \"inquiry/{id}\", etc)
   - HttpClientFactory pipeline includes TokenRefreshHandler

2. Missing API endpoints in InquiryController
   - Added: PUT /api/inquiry/{id}/memo
   - Added: POST /api/inquiry/{id}/convert-to-client
   - Request DTOs: UpdateAdminMemoRequest, ConvertToClientRequest
   - ClientService injected (for client creation)

**Implementation:**

- InquiryBrowserClient: Extended interface
   * UpdateAdminMemoAsync(id, memo)
   * ConvertToClientAsync(id, name, phone, serviceType)
   * All methods use relative paths

- InquiryBrowserClient.ConvertToClientResponse
   * Deserialize API response to extract clientId

- InquiryDetail.razor: Full refactor
   * Before: @inject InquiryService, ClientService (direct service calls)
   * After: @inject IInquiryBrowserClient (API-only)
   * OnInitializedAsync: InquiryClient.GetByIdAsync
   * OnStatusChanged: InquiryClient.UpdateStatusAsync
   * SaveMemo: InquiryClient.UpdateAdminMemoAsync
   * ConvertToClient: InquiryClient.ConvertToClientAsync

**InquiryList.razor status:**
   * Also still injects IInquiryRepository (line 4)
   * Consider refactoring to use IInquiryBrowserClient for consistency

**Phase 7 Status:**
- ✅ Blog page: Already API-First (ApiClient)
- ✅ Inquiry page: Fully API-First (IInquiryBrowserClient)
  * InquiryTable: ✅ Migrated
  * InquiryDetail: ✅ Migrated
  * InquiryList: ⏳ Still uses IInquiryRepository (minor - reads only)

**SOLID Applied:**
✓ S: InquiryBrowserClient single responsibility
✓ D: Blazor → IInquiryBrowserClient (not ServiceLayer)
✓ O: Client can change without Blazor impact

Next: Check FAQ, Client, TaxFiling pages for same pattern.
If all still injecting services directly, migrate sequentially.
Then: Phase 6 (SignalR) will have all pages ready.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
 2026-06-28 10:59:02 +09:00
kjh2064 e3f548f163 feat: include inquiry status changer in alerts
TaxBaik Browser E2E / browser-e2e (push) Failing after 1m6s
Details
TaxBaik CI/CD / build-and-deploy (push) Successful in 1m26s
Details
2026-06-27 16:36:31 +09:00
kjh2064 28060b71be feat: harden auth ops and deployment baseline 2026-06-27 10:53:53 +09:00
kjh2064 e22cfb1ac5 feat: REST API 계층 추가 - 완벽한 MVC/API 분리
TaxBaik CI/CD / build-and-deploy (push) Failing after 43s
Details 
4개 API 컨트롤러 구현:
✅ AuthController: POST /api/auth/login
✅ BlogController: GET/POST/PUT/DELETE /api/blog
✅ CategoryController: GET/POST/PUT/DELETE /api/category
✅ InquiryController: POST/GET/PUT /api/inquiry

아키텍처 개선:
- Application 서비스 레이어 확장 (CategoryService 추가)
- Repository 인터페이스 CRUD 지원 추가
- Program.cs에 MapControllers() 추가
- 비즈니스 로직과 UI 완전 분리

장점:
- 향후 UI 리뉴얼 시 API 변경 불필요
- 모바일 앱, 데스크톱 클라이언트 추가 가능
- 테스트 가능한 API 엔드포인트

테스트 결과:
✅ 블로그 API: 5개 포스트 조회
✅ 카테고리 API: 5개 카테고리 조회
✅ 문의 API: 문의 제출 성공
⚠️ 인증 API: 예정된 수정 대기

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
 2026-06-26 22:52:48 +09:00