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

文字列としてのリアクションを廃止 #5501

Closed
syuilo opened this issue Oct 13, 2019 · 9 comments · Fixed by #5712
Closed

文字列としてのリアクションを廃止 #5501

syuilo opened this issue Oct 13, 2019 · 9 comments · Fixed by #5712
Labels
💬Discussion Being discussed or needs discussion ✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR
Projects
Milestone

Comments

@syuilo
Copy link
Member

syuilo commented Oct 13, 2019

Summary

現在のMisskeyは

  • 文字列としてのリアクション
  • 絵文字としてのリアクション
  • カスタム絵文字としてのリアクション

があるけど、文字列を廃止して絵文字かカスタム絵文字かだけにしたい
分かりやすくなり、コードもすっきりする
プリンが二重に表示される問題(?)もなくなる
あと例えば'like'が来たら内部的に'👍'に変換してから処理すればいいので破壊的変更にはならない

@syuilo syuilo added ✨Feature This adds/improves/enhances a feature packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR labels Oct 13, 2019
@mei23
Copy link
Contributor

mei23 commented Oct 13, 2019

擬態プリン (pudding) と 真プリン (🍮) を区別していたのがなくなってしまう
真プリン (🍮) を勝手に🍣にしていいものなのか

プリンが二重に表示される問題(?)もなくなる

擬態プリンと真プリンが同じ用に表示されるのは仕様のつもり…

@syuilo
Copy link
Member Author

syuilo commented Oct 13, 2019

あー、プリンよりも寿司が好きオプションが動作しなくなる問題がある

@syuilo
Copy link
Member Author

syuilo commented Oct 13, 2019

プリンよりも寿司が好きオプションも廃止した方が良さそう

@mei23
Copy link
Contributor

mei23 commented Oct 13, 2019

えー

@syuilo
Copy link
Member Author

syuilo commented Oct 13, 2019

ほむ

@syuilo syuilo added the 💬Discussion Being discussed or needs discussion label Oct 13, 2019
@EbiseLutica
Copy link
Member

正直絵文字の仕様変更でプリン派も寿司派も仲良しになれたので、寿司好きオプション廃止は賛成です

@syuilo
Copy link
Member Author

syuilo commented Jan 13, 2020

v12でやります

@syuilo syuilo added this to the v12 milestone Jan 13, 2020
@syuilo syuilo added this to To do in v12 Jan 13, 2020
@mei23
Copy link
Contributor

mei23 commented Jan 13, 2020

DB上で既存投稿のリアクションが

{
  'like': 2,
  'love': 1
}

で集計されてるところに

{
  'like': 2,
  'love': 1,
  '👍': 1
}

みたいなのが入って来ますが、マイグレとかする予定です?

@syuilo
Copy link
Member Author

syuilo commented Jan 13, 2020

出来ればしたいですが、面倒そうなのでとりあえずはAPIレスポンス返す時にその都度 like だったら 👍 に変換するとかになりそうです

@syuilo syuilo moved this from To do to In progress in v12 Jan 14, 2020
@syuilo syuilo moved this from In progress to Done in v12 Jan 18, 2020
@syuilo syuilo mentioned this issue Jan 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💬Discussion Being discussed or needs discussion ✨Feature This adds/improves/enhances a feature packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR
Projects
No open projects
v12
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants