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

feat: Reaction on comments #9050

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tokiwa-t
Copy link

@tokiwa-t tokiwa-t commented Aug 26, 2024

社内でGROWIを利用しています。コメントに対するいいね!機能が欲しくなり、実装案としてPull requestします。

実装した画面

一部黒塗りで隠しています。

image

機能

  • 各commentに liker 属性を持たせ、ユーザーのリストを保持するようにしました
  • フロントエンドからいいね!するユーザーのIDを引数にAPIを呼び出し、バックエンドでいいね!の付与・剥奪の処理を行います
  • コメント更新、削除の実装を参考にしました。また、ページに対するいいね!の実装も参考にしています。

できていないこと

  • テストは実装できていません(ビルドして目視での動作は確認)
  • イベントログへの出力は対応できていません
  • SCSSがページに対するいいね!ボタンと重複するかもしれません

Copy link

changeset-bot bot commented Aug 26, 2024

⚠️ No Changeset found

Latest commit: 819d73c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@yuki-takei
Copy link
Member

yuki-takei commented Aug 27, 2024

@tokiwa-t すばらしい PR をありがとうございます!

機能の仕様について

ページ自体に liker があるのでコメントも同様にするのは順当ではありますが、
Mattermost や Misskey のような後発の SNS では thumb up だけでなく絵文字でリアクションできるようになっています。
X(旧Twitter)も今後絵文字でのリアクションに対応するとか…

一方、Misskey の絵文字リアクションはうるさいと感じるユーザーもいるようです。
GitHub は8種類に制限されていますね。

リアクションの種類をどうするかはちょっと考えたいところです。

@satof3 UXデザインの観点から意見あったらお願いします。

コードについて

現状のコードについては一通りレビューしましたが、とても完成度が高く修正すべき点は見られませんでした。
仕様を現状のままで確定するならこのままマージできると思います 🙂

@tokiwa-t
Copy link
Author

tokiwa-t commented Aug 28, 2024

@yuki-takei
レビューありがとうございます!
実装後の画面を本Pull requestの説明に追加しました。

仕様については、おっしゃる通り他の絵文字を使ってリアクションできると、コミュニケーションの幅が広がりそうですね。
個人的には、一旦いいね!ができるだけでも嬉しいので、今後段階的に選択できるようになっても良いかなと感じました。

ちなみに社内では、今回の機能は「コメントに返信するまででは無いけど読んだこと・感謝・称賛を伝えたい」という意図で使っています。

@yuki-takei
Copy link
Member

画面イメージありがとうございます。見た目に関してはそれこそリリース後にブラッシュアップでもよさそうですね。
機能的にも最初 👍 のみで段階的に別の絵文字が追加されるでもいいんですが、models/comment.ts の設計くらいは最初から最終系にしておきたいなという気持ちはありますね。

@yuki-takei yuki-takei changed the title add like button on comments feat: Reaction on comments Sep 2, 2024
@yuki-takei
Copy link
Member

Auto-labeling / check-title (pull_request) を通すために PR のタイトルを変えました

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

Successfully merging this pull request may close these issues.

2 participants