新規CIワークフローをトリガーするためにmainに対してドラフトのPRを作成 #1584
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* web-csrのbackendにweb-adminプロジェクトとweb-consumerプロジェクトを追加 * web-adminとweb-consumerの共有部分をwebに移動 * web-adminから不要なファイルを削除 * webにcontrolleradviceのテストメソッドを追加 * カタログアイテムのCRUD処理を行うリポジトリインターフェースと実装クラスの追記 * MyBatisを用いたCRUD処理を自動生成されたコードに変更 * ロガーや認証認可を除いたCRUD処理の実装 * APIの定義に合わせてCRUD処理と関連する箇所を修正 * 認証・認可関連のクラスを実装 * 認可をPreAuthorizeアノテーションで管理するように変更 * 開発環境でのみダミーユーザが注入されるように変更 * ユーザのセッション情報を管理するクラスを追加 * ログ出力の中身の詰め込みとそれに伴う例外クラスの追加 * 楽観同時実行制御用のrow_versionカラムをcatalog_itemsに追加し、myBatis generatorで生成したクラスを追加 * 認証認可に関するアノテーションの位置と種類を変更 * 楽観同時実行制御の実装 * コメントアウトやファイルの場所の変更 * 楽観ロックエラーの実装とコントローラのAPI定義の修正 * CatalogManagementApplicationServiceの単体テストの追加 * controllerAdviceのフォルダ構成の変更 * webをadminとconsumerに分割した際に生じた差分（主にCORS構成）を修正 * 権限エラーと楽観ロックエラーのメッセージをプロパティファイルに定義 * 例外ハンドリングにアノテーションによる認証エラーと認可エラーを追加 * ログレベルを適切なものに変更 * 更新用のコマンドオブジェクトを削除 * フォーマットのルールを他メソッドに合うよう修正 * api定義の切り分け * CRUD処理をドメインサービスに集約 * 更新処理をMyBatis Generatorで生成されたプログラムを用いたものに変更 * h2データベースの起動設定の変更 * launch.jsonにadminとconsumerを追記 * h2サーバーの設定を行うコンポーネントのDIをnull許容に変更する * 単体で使用しないためlaunch.jsonからwebを削除 * 単体テストのメソッド名の命名を統一 * プログラムの過不足の適用、ドメインサービスの更新処理のメソッド名を変更 * ユーザのセッション情報の実装クラスのメソッドを適切なものに修正 * DummyUserInjectionFilterのJavaDocコメントを追記 * 楽観同時実行制御のバージョンカラムの型をDBではTIMESTAMP型、JavaではLocalDateTimeに変更 * MyBatis Generatorで生成したプログラムを楽観同時実行制御対応させるためのタスクを追加 * 自動生成したマッパーを楽観ロック対応させるタスクにおいて列やディレクトリの柔軟性を向上 * h2サーバー起動用のクラスをインフラから削除し、adminとconsumerに新規追加 * CatalogManagementApplicationServiceとCatalogApplicationServiceに統合 * myBatisGeneratorConfigの誤字を訂正 * 楽観ロックエラーと権限エラーを共通エラーハンドラでキャッチするように変更 * H2ServerConfigをH2ServerLauncherに変更 * CatalogApplicationServiceのgetCatalogItemsメソッドの処理をprivateの共通処理からadminとconsumerで直接記述するように変更 * カタログアイテムの取得処理に認可チェックを追加 * コントローラの認可チェックを全てのアクションメソッドに付与するよう変更し、併せてAPI定義も修正 * APIのdescriptionの更新に伴うAPI仕様書の更新
prodビルド用のコマンドがワークスペースになかったため、追加
This reverts commit 389549b.
This reverts commit e001e14.
…into develop/admin/フロントエンドアプリケーションの開発
わかりにくい箇所を修正
* カタログアイテムのdtoの引数を修正 * アプリケーションサービスの更新メソッドの引数を修正 * カタログアイテムのgetのdtoのフィールドに行バージョンを追加 * API仕様の軽微な修正 * javadocの誤記を修正 * PagedListOfCatalogItemResponseのフィールドをMarisに合わせて修正 * カタログアイテムの削除処理と更新処理のアクションメソッドの引数を修正 * PagedListOfCatalogItemResponseのtotalPagesの計算を商と余りを用いて行うよう変更
…' into develop/admin/フロントエンドアプリケーションの開発
* カタログアイテムのdtoの引数を修正 * アプリケーションサービスの更新メソッドの引数を修正 * カタログアイテムのgetのdtoのフィールドに行バージョンを追加 * API仕様の軽微な修正 * javadocの誤記を修正 * 楽観ロック対応のタスクの不要な記述を削除 * ユーザロールを複数返すように変更 * DTOクラス群の命名を変更 * カタログアイテムを削除に楽観同時実行制御を実装 * 不足しているAPIレスポンスを追加 * ユーザーロールを定数で管理 * PagedListOfCatalogItemResponseの変更分をマージ * adminのAPI仕様書を更新 * adminのPagedListOfCatalogItemResponseをPagedListOfGetCatalogItemResponseに変更 * adminとconsumerのdependabot version updateの設定を追加 * 指摘事項への対応
…' into develop/admin/フロントエンドアプリケーションの開発
|
本来のPRが作成され、役割を終えたためクローズします。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
この Pull request で実施したこと
この Pull request で何をしたのか記載します。
この Pull request では実施していないこと
この Pull request と関連する機能やドキュメントに対して行わなかったことを記載します。
ない場合は「なし」で構いません。
既に Issues 等が整備できている場合は、リンクを記載します。
Issues や Discussions 、関連する Web サイトなどへのリンク
関連する Issues や Discussion へのリンクを記載します。
また、このプルリクエストを作成するにあたり参照した資料へのリンクを記載します。