Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dressca のフロントエンドについて Maia v1.0 の変更を移植 #2269

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

KentaHizume
Copy link
Contributor

@KentaHizume KentaHizume commented Jan 15, 2025

この Pull request で実施したこと

確認ポイント

  • dressca-frontend 配下のディレクトリに想定外の差分がないことを確認しました。
  • (consumer) 買い物かごにアイテムを追加~注文のケースが正常に実行できることを確認しました。
  • (consumer) API コールでエラーが発生した際に、エラーを示すトーストが表示されることを確認しました。
  • (admin)カタログアイテムに対するCRUD操作が正常に実行できることを確認しました。

あるべき差分

  • src/generated 配下の自動生成された API クライアントコード
  • 各ワークスペース配下のpackage.json の OpenAPI 仕様書のパス
  • consmer ワークスペース配下のpackage.jsonvite previewのポート番号
  • env.devで指定するバックエンドAPIのポート番号

この Pull request では実施していないこと

vite previewのポート番号については下記のissueでmaia/marisともに指定なし(4173)に統一します。

Issues や Discussions 、関連する Web サイトなどへのリンク

下記の解決により、v0.9までは存在したconsumerのmockの差分が解消されました。

下記の対応により、APIコールの失敗時にmaia側ではレスポンスのProblemDetailsから詳細なメッセージをトースト通知しますが、maris側はバックエンド側が未対応のため、既存と変わらない出力を行います。

@KentaHizume KentaHizume added this to the v1.0.0 milestone Jan 15, 2025
@KentaHizume KentaHizume linked an issue Jan 15, 2025 that may be closed by this pull request
2 tasks
@KentaHizume KentaHizume added the target: Dressca サンプルアプリケーションDresscaに関係がある label Jan 15, 2025
@KentaHizume KentaHizume self-assigned this Jan 15, 2025
@tsuna-can-se tsuna-can-se merged commit c35dee2 into main Jan 15, 2025
7 checks passed
@tsuna-can-se tsuna-can-se deleted the feature/(Dressca)maiav1.0の変更を移植 branch January 15, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v1.0 向けのフロントエンドアプリの移植
2 participants