search

mercari-build / mercari-build-training

24 stars 225 forks source link

Third pull request #186

Closed shiori-42 closed 8 months ago

shiori-42 commented 8 months ago

What

CHECKS :warning:

Please make sure you are aware of the following. STEP4の実装とファイルを分割しました。ご確認よろしくお願い致します。

shiori-42 commented 8 months ago

コードを書き換えました。JSONのエンドポイントは削除しました。確認用に出力していたデータベース情報はいただいたアドバイスのログ出力の実装をしてみましたが、かなりコードが長くなってしまったため重要性を鑑みて削除しました。 構造体はItemのみに書きかえたのですが、出力時にcategoryだけでなく、categoryIDも出力してしまいますが、こちらは問題ありませんか？categoryIDで取得したcategoryのみを出力するのかなと解釈していたので質問させていただきます。

shiori-42 commented 8 months ago

このブランチのままdockerfileの作成まで行いました。エンドポイントは全てコンテナを起動してテストして問題なさそうでした！とっても嬉しいです！お忙しいところ恐れ入りますがご確認よろしくお願いいたします。

momom-i commented 8 months ago

@shiori-42 すみません、ご自身がフォークしたレポジトリでPR作成していただいても良いでしょうか？ 🙏 一旦クローズします