Skip to content

Create main.md#3

Open
yuji91 wants to merge 1 commit into
masterfrom
add_main
Open

Create main.md#3
yuji91 wants to merge 1 commit into
masterfrom
add_main

Conversation

@yuji91

@yuji91 yuji91 commented May 6, 2021

Copy link
Copy Markdown
Contributor

完成形のGIFやHerokuでの完成版アプリの紹介は要らないですかね?

不足部分があれば指摘をお願いしますm(_ _)m

@DaichiSaito

Copy link
Copy Markdown

完成形のGIFやHerokuでの完成版アプリの紹介は要らないですかね?

あったほうがいいっすね!

Comment thread main.md
- ユーザーログアウト機能
- 投稿編集・削除機能の制限

実装では課題と同様`sorcery`のgemを使用して、各機能ごとに以下の観点を満たすように実装してください。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

細かい文言ですが、「観点」より「ポイント」の方がいいかも
下の方も同じく「確認観点」→「確認ポイント」
判断はいとうさんに任せます:pray:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

「確認観点」はテストコードのitの文章に対応した名前なので、そっちも直さないといけなくなっちゃうんですよね、、

最初の「観点」だけ「ポイント」に変えておこうと思います!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あ、でも「確認観点」が変わらないのであれば「ポイント」に変えなくてもいい気がしてきました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants