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

カスタム絵文字が複数登録できなくなった #10996

Closed
fruitriin opened this issue Jun 12, 2023 · 65 comments · Fixed by #13473 or tiramiss-community/tiramiss#8
Closed

カスタム絵文字が複数登録できなくなった #10996

fruitriin opened this issue Jun 12, 2023 · 65 comments · Fixed by #13473 or tiramiss-community/tiramiss#8
Assignees
Labels
⚠️bug? This might be a bug [Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features.

Comments

@fruitriin
Copy link
Contributor

fruitriin commented Jun 12, 2023

💡 Summary

🥰 Expected Behavior

🤬 Actual Behavior

📝 Steps to Reproduce

📌 Environment

Misskey version:13.13.1
PostgreSQL version:
Redis version:
Your OS:
Your browser:

@fruitriin fruitriin added the ⚠️bug? This might be a bug label Jun 12, 2023
@fruitriin
Copy link
Contributor Author

週に50個とか登録してるのでめちゃくちゃ困ってます

@syuilo
Copy link
Member

syuilo commented Jul 4, 2023

複数登録できないというのがよくわかっていない

@tamaina
Copy link
Contributor

tamaina commented Jul 4, 2023

前はファイル複数選択で一気に登録できた気がする

@icalo35
Copy link

icalo35 commented Jul 10, 2023

v13.12.2までは「絵文字を追加」>アップロード、で複数のファイルを選択して一括でアップロード出来て、ファイル名からデフォルトで名前が付けられていました。
なので、絵文字登録を依頼するユーザに自分でファイル名を付けて貰ってさえいれば、管理者はzipで受け取ったファイルをまとめて放り込む形で10個でも20個でも一気に登録することが出来ました。

現在(v13.13.2)は「絵文字を追加」を押すとNew emojiダイアログが表示され「ファイルを選択」>アップロードという操作ですが、ここでは1つのファイルしか選択する事が出来ません(恐らくダイアログで扱える絵文字が1つだけのため?)
絵文字が20個あれば20回、開いて選択して登録して、という操作を繰り返す必要があります。
しかも、ファイル名がデフォルトの名前としても採用されず空白のまま(#10997)なので毎回名前をコピペする操作が必要です。

ということで、手間と労力が激増して非常に困っております。

@prlg25
Copy link

prlg25 commented Jul 19, 2023

最近13.13にアップデートしたのですが、まとめて登録できないので13.12以前と同じ運用ができなくなり非常に困っています。
ユーザーからカスタム絵文字の申請が週に数十~100以上のペースであるサーバーの場合、カスタム絵文字をまとめて登録する機能がないととてもではないですが運営していけません。
ほんとに無理です。たすけてください。

@tai-cha
Copy link
Contributor

tai-cha commented Jul 19, 2023

一応代替策としてmeta.json作ってzipにまとめてインポートから登録すれば一括で登録自体はできるような気はします

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

meta.json無くてもzip importできるようにすれば解決するわね

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

もしくはまとめて登録用の選択肢追加するとか

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

zipインポートの方が手っ取り早そう

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

現状のzipインポートは失敗率が高く、そしてその原因もいまいちわからないので、何か新たな仕組みを用意した方が良い感じがしてます。

ウチも割と申請多いので対策がほしいところ……。何卒。

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

zip importが失敗することも直って欲しいしそれとmeta.jsonが不要になれば解決するような気がします…?

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

zipインポートはサイズの問題では…?

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

zip importが失敗することについてのIssueどこ?

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

#10778 なるほど

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

当時絵文字のバルクインポート自体がライセンスの観点で問題視
#10778 (comment)

そうなの

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

meta.jsonがなくなるというのも一案だけども、結局Web UIでの負荷が大きいと大量登録の負荷は変わらない印象があったりもする...という気持ちです。
meta.jsonは「オプションファイル」という感じで処理できたら一番良いのかなと。

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

失敗したときにUI側でエラー表示できるかどうかというのもあった方が良さそうですね...。
(今はエラーが出ないので上手くいったと思い込んでしまう。)

@tamaina
Copy link
Contributor

tamaina commented Jul 20, 2023

絵文字のzipインポートのAPIとしての動作はサーバーのキューに突っ込むだけなので、キューの実行中に出たエラーをフロントエンドに出すのはかなり難しいかもです

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

失敗したら管理者にメールで通知するとかが一番簡単そう

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

Web UIだと気づきやすいというのはあるのですが、何かしらの通知が来るだけでもかなりの改善になる気がしますね。

@CyberRex0
Copy link
Contributor

コントロールパネルにログビューア作るとか

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

かなり脱線するけどユーザー個別に管理者/システムからお知らせ送れる機能は欲しいと思ってる

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

ちょっと脱線するけどmeta.jsonからライセンスも登録したいです

(あれ、いまできなかったですよね)

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

こんなのがある
#10221

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

1個1個の追加はめっちゃ手間なのでとりあえずでzipとかの手段があるとありがたい気持ちはめっちゃあります

某絵文字94個パック入れたときもunzip失敗してapplication/octet-streamのゴミファイルが登録されたやつの差し替え(とファイル削除)がちょっと大変だった

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

ユーザーからカスタム絵文字の申請が週に数十~100以上のペースであるサーバーの場合、カスタム絵文字をまとめて登録する機能がないととてもではないですが運営していけません。

個別に登録だと何が困るのかしら
普通、ユーザーから申請されたとするとひとつひとつ(著作権的に大丈夫かなどの)審査をして登録するだろうから「一度にまとめて何十個も登録する」っていうシチュエーションがあまり想像できてない

@acid-chicken
Copy link
Member

どちらかというと今の個別登録の操作が煩雑すぎる

@syuilo
Copy link
Member

syuilo commented Jul 20, 2023

絵文字追加ボタン押す→ファイル選択する→OK押す
だと思うけどこれ以上簡略化できるかしら

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

現状は
・絵文字の名前にファイル名が反映されないので入力
・カテゴリーを選択する
・タグを入力する(エイリアス登録)
の手順を全て個別でやる必要があって、先に審査が終わっているものをまとめて登録したいときにはものすごい手間になっているのは事実だったりします。

@samunohito
Copy link
Member

表部分の動作イメージ
mkgrid-sample

@syuilo
Copy link
Member

syuilo commented Jan 26, 2024

すごそう

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 26, 2024

スマホでの動作をどうするかが懸念点かも
(まあそもそもExcel UI非対応にするのもアリ)

@samunohito
Copy link
Member

表示についてはデッキ表示みたいにあえてレスポンシブにせず、スクロール対応で良い気がしています。
無理に詰め込むと却って視認性が落ちてしまいますし、実装が超大変なので
操作面については、クリック・ダブルクリックなどの操作をスマホでやりやすいような形(具体的にはまだ考えていない)にしてもいいかと思います。

@samunohito
Copy link
Member

以下でまとめてくださっている事項については解決できるかと考えます。
#10996 (comment)
#10996 (comment)

方針的によさそうであれば引き続き実装を進めますが、いかがでしょうか。

@syuilo
Copy link
Member

syuilo commented Jan 26, 2024

表コンポーネントに関しては自前で用意する感じかしら

@samunohito
Copy link
Member

自前で作ってます

@syuilo
Copy link
Member

syuilo commented Jan 26, 2024

考慮しないといけない点が多そうでなかなか難しそうだけど出来そうならよさそう

@samunohito
Copy link
Member

(既存の画面は残しつつ、表形式の画面をベータ版として別に実装し、安定した段階で完全に置き換えるのもアリだと思います)

@samunohito samunohito self-assigned this Jan 26, 2024
@samunohito
Copy link
Member

まだまだやるべきことはありますが、最低限動くようになったので今の様子を上げておきます
grid-prog1

@syuilo
Copy link
Member

syuilo commented Jan 29, 2024

おー

@samunohito samunohito moved this to In Progress in [FEATURE] Admin Jan 30, 2024
@samunohito
Copy link
Member

samunohito commented Feb 7, 2024

今の状態








Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug [Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features.
Projects
Status: In Progress