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

コメント機能を実装しました! #100

Merged
merged 12 commits into from
Jan 15, 2021
Merged

Conversation

Kattsuuya
Copy link
Collaborator

変更内容

匿名のコメントを書けるようにしました!
#96 とコンフリクトするかもだけど、機能面だけ実装したから許して😅

image

Issue

close #64

@mei28
Copy link
Member

mei28 commented Jan 13, 2021

見栄えとか,再描画のところ直したいけど,直せないから助けて欲しい,,,

Comment on lines 163 to 165
isNoComment() {
return this.inputComment === "";
},
Copy link
Member

Choose a reason for hiding this comment

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

これでも動くけど常に判定し続けるものはcomputedに書くのがセオリーな感じがする!!

@hotsukai
Copy link
Member

hotsukai commented Jan 13, 2021

見栄えとか,再描画のところ直したいけど,直せないから助けて欲しい,,,

bb9e9bc で対応しました

@hotsukai
Copy link
Member

hotsukai commented Jan 13, 2021

  • 新しいコメントほど上に表示されたほうがいいかも
  • 投稿時間も表示されると嬉しかも

@mei28
Copy link
Member

mei28 commented Jan 13, 2021

  • 新しいコメントほど上に表示されたほうがいいかも

時間がでてないからわかりづらいけど,上の方が最新になっているはず

  • 投稿時間も表示されると嬉しかも

投稿時間の取得方法がわからなかった!

@hotsukai
Copy link
Member

hotsukai commented Jan 13, 2021

新しいコメントほど上に表示されたほうがいいかも

これは自分の bb9e9bc でのミスでした🤦‍♂️

投稿時間も表示されると嬉しかも

@mei28 やりました!

@nanaka0012
Copy link
Member

ローカルでは確認してなくて、PRのpreview環境なんだけど、
なんか新しくコメントしたやつが下に潜っちゃう: (

image

@mei28
Copy link
Member

mei28 commented Jan 13, 2021

image
自分はとくに下に潜らない.
かいほつのが残ってた説.

@Kattsuuya
Copy link
Collaborator Author

image

@hotsukai
Copy link
Member

image
自分はとくに下に潜らない.
かいほつのが残ってた説.

これなんで秒まで表示されてるんだろう??

@Kattsuuya
Copy link
Collaborator Author

色々直した!!
レビューおなしゃす!!

Copy link
Member

@nanaka0012 nanaka0012 left a comment

Choose a reason for hiding this comment

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

image

Copy link
Member

@mei28 mei28 left a comment

Choose a reason for hiding this comment

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

いいとも

@Kattsuuya Kattsuuya merged commit f350936 into develop Jan 15, 2021
@Kattsuuya Kattsuuya deleted the feature/comments branch January 15, 2021 05:32
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.

記事に対してコメント(匿名?)をつけたい!
4 participants