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.

Sign up for GitHub

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

Jump to bottom

コメントをVue.js化 #1017

Merged
merged 18 commits into from Jul 10, 2019
Merged

コメントをVue.js化 #1017

merged 18 commits into from Jul 10, 2019

Conversation

komagata
Copy link
Member

No description provided.

@komagata komagata changed the title commentのvueを追加 コメントをVue.js化 Jun 13, 2019
@komagata komagata temporarily deployed to bootcamp-fjord-jp-pr-1017 June 13, 2019 06:59 Inactive
@komagata komagata temporarily deployed to bootcamp-fjord-jp-pr-1017 June 21, 2019 09:07 Inactive
@kasumi8pon
Copy link
Contributor

@machida
デザインをよろしくお願いいたします！

デザインをお願いしたい変更点は以下の2点です

  • コメント編集ボタンを押した際に出てくるフォーム下部の保存するボタンとキャンセルボタン(comment.vue:41)
  • 新規コメントとコメント編集フォームで何も入力せずに投稿した際に表示されるエラーメッセージ(comments.vue:21 comment.vue:36)

@machida
Copy link
Member

machida commented Jun 25, 2019

了解です！

@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 June 26, 2019 08:07 Failure
@kasumi8pon
Copy link
Contributor

@machida
エラーメッセージの表示を消しました。
コメント編集ボタンを押した際に出てくるフォーム下部の保存するボタンとキャンセルボタン（comment.vue:40）部分のデザインをお願いします！

@machida machida self-assigned this Jul 3, 2019
@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 3, 2019 04:40 Failure
@machida
Copy link
Member

machida commented Jul 3, 2019

@kasumi8pon お待たせしました🙇‍♂️デザイン入れました！

@machida machida removed their assignment Jul 3, 2019
@kasumi8pon kasumi8pon marked this pull request as ready for review July 3, 2019 06:43
@kasumi8pon
Copy link
Contributor

@machida デザインありがとうございます！

@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 3, 2019 07:30 Failure
@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 3, 2019 08:54 Failure
@kasumi8pon
Copy link
Contributor

@komagata
レビューよろしくお願いします！
（駒形さんにPRを作成していただいたので、駒形さんにレビューのリクエストができないようです。
そのため、コメントのみで依頼させていただきます。）

komagata
komagata commented Jul 3, 2019
View reviewed changes
app/controllers/api/comments_controller.rb Outdated Show resolved Hide resolved
app/controllers/concerns/commentable_controller.rb Outdated Show resolved Hide resolved
app/views/api/available_emojis/index.json.jbuilder Outdated Show resolved Hide resolved
@komagata
Copy link
Member Author

komagata commented Jul 3, 2019

@kasumi8pon 自分へのコメントになっちゃってるんですが、レビューとしてコメントしましたー

@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 7, 2019 05:37 Failure
@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 7, 2019 08:15 Failure
@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 7, 2019 08:41 Failure
@kasumi8pon
Copy link
Contributor

@komagata
失礼しました。スタイルなど修正しました。
あわせて、checkableとwatchableをcommentableと同じ方法で取得するよう変更しました。
再度レビューをお願いいたします。

@komagata komagata had a problem deploying to bootcamp-fjord-jp-pr-1017 July 8, 2019 08:34 Failure
@kasumi8pon
Copy link
Contributor

@komagata
インデント等修正いたしました🙇‍♀️
再度よろしくお願いします。

@komagata
Copy link
Member Author

@kasumi8pon 確認しました、OKですー🙆‍♂️
（自分でPR作ってしまったのでレビューOKがおせないみたいです、すいません）

@kasumi8pon
Copy link
Contributor

@komagata
レビューありがとうございます！マージします。

@kasumi8pon kasumi8pon merged commit ebc69a9 into master Jul 10, 2019
@komagata komagata deleted the comment-to-vue branch February 10, 2021 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@komagata @kasumi8pon @machida