質問・回答アプリ（テーブル設計編)の課題提出 #1
Merged
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
DaichiSaito
reviewed
Nov 28, 2021
design_table.md
Outdated
| - id: integer | ||
| - name: string | ||
| - email: string | ||
| - password: string |
There was a problem hiding this comment.
passwordだと生のパスワードを保存するように見えてしまいます。
また、has_secure_passwordモジュールを利用する場合はpassword_digestという名前である必要があります。
以上のことからpassword_digestにすべきかなと思います。
DaichiSaito
reviewed
Nov 28, 2021
design_table.md
Outdated
Comment on lines
16
to
21
| - notifications | ||
| - id: integer | ||
| - question_id: integer | ||
| - visitor_id: integer | ||
| - visited_id: integer | ||
| - checked: integer |
There was a problem hiding this comment.
これってどういう用途のテーブルでしょうか？
回答とはまた別でしょうか？
There was a problem hiding this comment.
通知のテーブルを作成しました。
ですが、少々修正を加えました。
There was a problem hiding this comment.
通知って今回の要件にはないと思うんですが、作った意図はなんでしょうか？
また、「回答」がないのが気になります。
There was a problem hiding this comment.
課題要件に
質問があった際に全員に対して質問があった旨をメールで通知する（ただし自分は除く）
質問に対して回答があった場合は質問者および当該質問に回答したユーザーに対してメールで通知する。（ただし自分は除く）
とあり、Railsで通知機能を作る（①基本機能編）を参考に作りました。
ですが、今回削除しました。
There was a problem hiding this comment.
なるほど。
「メールで通知する」なのでDBに保存する必要はないかなと思います。テーブルを作っても良いとは思いますが、それよりもやはり「answers」を作るという発想が一般的かなと思います。
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.
質問・回答アプリ（テーブル設計編)の課題提出
テーブル設計のファイルを
design_table.mdに書きました。ご確認お願い致します。
蛇足ですが、練習がてらER図も作成しました。
