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

open-apiでコード生成に利用するyamlを定義し、コントローラ部のUTと実装まで行う #38

Merged
merged 42 commits into from
Oct 5, 2023

Conversation

rami2076
Copy link
Owner

@rami2076 rami2076 commented Oct 4, 2023

ISSUE

対応内容

  • バリデーション要件決め
    • 文字数
    • 文字種
    • ドキュメントにまとめる
  • GitHubTemplateを追加
    • 動作確認
  • openapiのyamlを設定して5つの正常形が動作するように変更する
  • integration testに5つのエンドポイントのテストを追加する.(20xのみ)
  • openapiでエラー判定のユースケースを設定する
  • openapiでエラーハンドラーの設定を行う.もしくは自前のエラーハンドラを設定する.
  • 50xのITの追加を行う
  • 40xのUTの追加を行う
  • domain-modelの作成

詳細はコードをご確認お願いします。

動作確認

レビューコメントについて

ラベル 意味 意図
Q 質問 (Question) 質問。相手は回答が必要
FYI 参考まで (For your information) 参考までに共有。アクションは不要 (確認事項を残す場合などに使用)
NITS あら捜し (Nitpick) 重箱の隅をつつく提案。無視しても良い
NR お手すきで (No rush) 今やらなくて良いが、将来的には解決したい提案。タスク化や修正を検討
IMO 提案 (In my opinion) 個人的な見解や、軽微な提案。タスク化や修正を検討
MUST 必須 (Must) これを直さないと承認できない。修正を検討

https://zenn.dev/hacobell_dev/articles/code-review-comment-prefix

その他

@rami2076 rami2076 changed the title open-apiでコード生成に利用するyamlを定義する open-apiでコード生成に利用するyamlを定義し、コントローラ部のUTと実装まで行う Oct 5, 2023
@rami2076 rami2076 merged commit ac903f2 into main Oct 5, 2023
1 check passed
@rami2076 rami2076 deleted the feature/15 branch October 5, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant