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

リアクションビューワーから絵文字をインポートしようとするとエラーが発生する #487

Closed
1 task done
1673beta opened this issue Aug 20, 2024 · 4 comments

Comments

@1673beta
Copy link
Contributor

💡 Summary

image
画像のようにリアクションビューワー上に絵文字が存在する場合、クリックしてインポートしようとするとNo such emojisが出る

🥰 Expected Behavior

インポートできる(admin/emoji/stealに正しくパラメータが伝わる)

🤬 Actual Behavior

インポートできない

📝 Steps to Reproduce

  1. リアクションビューワーからリアクションをインポートする

💻 Frontend Environment

* Model and OS of the device(s): macOS 14.6
* Browser: Vivaldi 6.8
* Server URL: c.koliosky.com
* CherryPick: 4.10.0-rc.3-engawa.0.5.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* CherryPick:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@noridev
Copy link
Collaborator

noridev commented Aug 21, 2024

CherryPick 4.10.0-rc.3バージョンを基準とするbeta.kokonect.linkでは、この問題が再現されていません。
CherryPickの改造バージョンではなく、修正が加えられていないCherryPickと新しいDBでテストを進めていただけますか?

@1673beta
Copy link
Contributor Author

CherryPick 4.10.0-rc.4で再現しました。
202408311131_cherrypick_test

@1673beta
Copy link
Contributor Author

1673beta commented Aug 21, 2024

CherryPick 4.10.0-rc.3バージョンを基準とするbeta.kokonect.linkでは、この問題が再現されていません。

ノートとインポートしようとしているリアクションが同一hostだと発生しないようですが(ノートのhostとMkReactionViewer.reaction.vueのprops.note.reaction.hostが一致するため)、それぞれが違うホストの場合発生しませんか?

@noridev
Copy link
Collaborator

noridev commented Oct 3, 2024

該当エラーが 06faf6c で修正されました。ありがとうございます!🙏

@noridev noridev closed this as completed Oct 3, 2024
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 a pull request may close this issue.

2 participants