Skip to content

refactor: API仕様更新とStaff機能の削除#28

Open
Atsumi3 wants to merge 2 commits intomainfrom
refactor/remove-staff-api
Open

refactor: API仕様更新とStaff機能の削除#28
Atsumi3 wants to merge 2 commits intomainfrom
refactor/remove-staff-api

Conversation

@Atsumi3
Copy link
Member

@Atsumi3 Atsumi3 commented Oct 3, 2025

概要

API仕様をv2に更新し、Staff/User関連機能を完全に削除しました。

変更内容

API仕様の更新

  • /api/staffエンドポイントを削除
  • /api/usersエンドポイントを削除
  • CreateSaleRequestからstaffBarcodeフィールドを削除
  • ErrorResponse.detailsをMap型からString型に変更(kotlinx.serialization互換性のため)

コード変更

  • Entity: Staff entityを削除
  • Event: BarcodeEventからStaff関連イベント(ReadStaffSuccess/Failed)を削除
  • Config: GlobalConfigからcurrentStaffプロパティを削除
  • API: APIServiceからgetStaff()メソッドとcreateSale()staffBarcodeパラメータを削除
  • DI: 本番・デモ両方のDI設定からStaffApi、UsersApiの依存を削除

UI変更

  • ItemListViewModelからStaff表示機能(LiveData、イベントハンドラ)を削除
  • fragment_item_list.xmlからStaff表示UIコンポーネントを削除
  • MainViewModelからStaff関連処理を削除

影響範囲

  • Staffバーコードスキャン機能は動作しなくなります(空実装)
  • Staff情報の表示UIが削除されます
  • 売上作成時にStaff情報が不要になります

テスト

  • ./gradlew assembleProdDebug 成功
  • ✅ APIクライアントコード再生成完了
  • ✅ ビルドエラーなし

統計

  • 11ファイル変更
  • 111行追加、365行削除

API仕様をv2に更新し、Staff/User関連機能を完全に削除しました。

## 主な変更内容

### API仕様の更新
- `/api/staff`エンドポイントを削除
- `/api/users`エンドポイントを削除
- `CreateSaleRequest`から`staffBarcode`フィールドを削除
- `ErrorResponse.details`をMap型からString型に変更(kotlinx.serialization互換性のため)

### コード変更
- Staff entityを削除
- BarcodeEventからStaff関連イベント(ReadStaffSuccess/Failed)を削除
- GlobalConfigから`currentStaff`プロパティを削除
- APIServiceから`getStaff()`メソッドと`createSale()`の`staffBarcode`パラメータを削除
- DI設定からStaffApi、UsersApiの依存を削除

### UI変更
- ItemListViewModelからStaff表示機能を削除
- fragment_item_list.xmlからStaff表示UIを削除
- MainViewModelからStaff関連処理を削除

## ビルド確認
- `./gradlew assembleProdDebug` 成功
- BarcodeKind enumを削除し、全てのバーコードを商品として扱うように変更
- BaseBarcodeReadableActivityのインターフェースを簡素化
- サーバー接続エラー時のエラーメッセージ表示を改善
- MainActivityとMainViewModelのバーコード処理ロジックを整理
- デバッグメニューからスタッフ用ダミー入力を削除
- OpenAPI仕様書(api.yaml)を追加
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant