-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat/backend/api/v1 #17
Conversation
Go詳しくないのでメモがてらテーブルを貼っておきます、すみません
https://entgo.io/docs/code-gen/#schema-description を参考に以下のコマンドで生成しました。 # typing-server/ ディレクトリで
go run -mod=mod entgo.io/ent/cmd/ent describe ./domain/repository/ent/schema |
PRのテンプレートを作ったのならあえて破壊する理由がない限り、沿った方がよいと思う |
OpenAPIからドキュメントを作るとこんな感じになります。これも併用しつつレビューします |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
おつかれさまです
typing-server/openapi.yaml
Outdated
type: number | ||
format: float | ||
required: true | ||
description: 正確性 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
正確性という言葉だけでなく計算方法も示してあげる方が親切かと。例えば正確性 (ミスタイプ数 / キーストローク数)
、など。これ書くならむしろ計算方法だけの方がいいかもですけどその辺は読みやすいように書いてください。
(杞憂だと思いますがfloatの丸め誤差がないと思うけど少し気になる&ミスタイプ数があればGo側で計算できるから、accuracyの代わりにミスタイプ数を送れば整数で送れて_HAPPY_かもしれません)
プログラムで使えるコードを自動生成するときに機械的に命名されて 変なクラス名などになるのを防ぐ目的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OpenAPIを見ました
description: 不正なリクエスト | ||
'404': | ||
description: スコアが見つからない | ||
"201": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
こちらも作成された Score を返すべきだと思います!根拠は RFC7231 です (これは MDN) https://developer.mozilla.org/ja/docs/Web/HTTP/Status/201
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
おまけでそのスコアだとおよそ何位になるかという情報も一緒に返してあげると嬉しいかもね(RESTとは)(これは今後の課題というか半分冗談なので対応しなくてよいです!) 👍
よ~しスピード優先でマージしちゃうぞ~(合意済み) |
やったこと
やらないこと