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

Open
fruitriin opened this issue Jun 12, 2023 · 65 comments · May be fixed by #13473
Open

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

fruitriin opened this issue Jun 12, 2023 · 65 comments · May be fixed by #13473
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

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

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

ファイル選択時にファイル名を絵文字名に自動入力すればマシになりそう

これだけでもかなり負荷は変わると思われる点ですね。

@sasagar
Copy link
Contributor

sasagar commented Jul 20, 2023

画面を大々的に改変することになるけども、表のようなスタイルで10個ずつくらい入れていけるようにするのはよろしくないのかなと頭をよぎりました。Just ideaですけど。

@acid-chicken
Copy link
Member

ファイル選択時にファイル名を絵文字名に自動入力すればマシになりそう

理想を言えば複数選択した時以降のファイルは前のファイルの登録が終わるまで待っていて、終わったら立て続けにダイアログが出てきて欲しい

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

タグ(エイリアス)の設定は多分同じの追加すること少ないと思うので個々でやる必要があるけど同じカテゴリやライセンスの一括追加ならselectモードから一括設定することで多少楽できるはなし(出てなかったので)

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

なのでファイル名が自動入力されないのが一番のボトルネックな気がします

@tai-cha
Copy link
Contributor

tai-cha commented Jul 20, 2023

Misskeyにはマルチウィンドウ機能(これすき)があるはずなので効率を重視してその辺り活用したら別タブに行かなくてもよかったりするなはずではあります()

カテゴリは部分的にでも名前を覚えていればサジェストされますし

@prlg25
Copy link

prlg25 commented Jul 20, 2023

あるべき姿と現状について、議論が錯綜していて分かりにくかったので私の考えで恐縮ですが整理します。

あるべき姿:100個くらいの大量の絵文字を登録するときに、楽に、早く登録し終えられる。
少なくとも登録が楽であってほしい。登録〜オプション設定まで全て楽だと最高。

13.12時点のフロー:
①絵文字登録:ファイル名を絵文字名として複数ファイルを一気にアップするだけ。楽に終わる。
②カテゴリー付け:複数選択を使って楽に終わる。
③タグ付け(エイリアス):1つずつやる。とても大変。必須ではないのでやらなくても良い

13.13時点のフロー:
①絵文字登録:個別選択&ファイル名を入力する。とても大変なのにやらないと絵文字が登録されない。(ここが超問題)
②カテゴリー付け:複数選択を使って楽に終わる。
③タグ付け:1つずつ行うので大変なのは変わらず

ここで重要な問題は2つあって、
①一度に1つずつしか絵文字が登録できない。
②絵文字名が自動入力されない。

両方とも解決されると、絵文字登録作業がスムーズになり、非常にありがたいです。
実装としては、今の個別登録のUIに複数登録画面を追加して以前の仕様を復活させる形が、案ですが簡便かと思います。
少し話がそれますが、個別登録の場合にも、絵文字名はファイル名から自動入力されるべきかと思います。

また、以前から不便ではあったが許容されていた問題として、以下があります。
①タグ付けなどのオプション設定の手間が大きい。もっと楽にやりたい。

この問題についても、テーブル形式のUIなどこちらのIssueで議論がなされていますが、微妙に話が違う気がしていて新しくIssue建ててもいいような気がしています。
カスタム絵文字登録がよりあるべき姿になるためには重要な議論だと思います。

@icalo35
Copy link

icalo35 commented Jul 20, 2023

私からも補足させてください。
【「ダイアログを開く→画像ファイルを選ぶ→登録する」という操作を絵文字の数だけ絶対に繰り返しやらされる」のがとても困る、という点が最大の問題です。
このIssueで元々問題視していたのはこの1点、操作量のみである認識です。オプションは別の話です。

オプション一切無しの絵文字1件を登録するのに
「5クリック→名前入力→1クリック」の操作が現状必要です。名前が自動入力されても6クリック要る。
絵文字を50件登録するのに合計300クリック分のマウス操作が必要ということです。
5分以上ひたすらポチポチするだけの単純作業が発生しますし、ミスして同じファイルを2回アップロードしてしまうとドライブから消す手間が発生するなどストレスフルです。定期的にやる必要があるのはつらい。
v13.12以前の複数登録が可能であれば、絵文字50件が7クリックとかで登録出来ます(現実には3,4回に分けると思いますがそれでも段違いにラク)

これって以前のようにカスタム絵文字複数登録できるようにしても結局は個別に編集必要じゃない?

いいえ。Select modeでまとめて設定出来るおかげで「個別に編集」する労力は格段に少なく済みます。
大量に絵文字を申請される場合「同じカテゴリのものをまとめて」「同じ申請者(≒絵文字作者)がまとめて」というケースが多いです。弊サーバーでは活発に絵文字を作成して申請する人がある程度決まってるという事情もありますが、同じような規模のサーバーなら似たような状況ではないかと想像します。
申請者が同じであればライセンスの記述も全く同じか少しの修正で使いまわし入力する余地が十分にありますし、カテゴリは見た目で判断出来るのでまとめて設定は簡単です。タグは……タグだけは個別でなんとかしてますが……

まずはオプションは空っぽでいいので、とにかく複数の画像ファイルを少ない操作で絵文字として登録さえ出来れば後はSelect modeの力でなんとかなっておりましたので、その手段さえあればという感じです。

@tai-cha
Copy link
Contributor

tai-cha commented Jul 21, 2023

13.14.1で#10780 の変更入ったっぽいので暫定的にzipインポートが使えそう

@samunohito
Copy link
Member

ExcelやGoogleスプレッドシートに近い(考えてる機能としてはIntelliJ系のDBテーブルエディタが更に近い)動きを実装すれば複数の絵文字登録および更新による手数を大幅に減らせると考えました。

以下は参考イメージです。

具体的には以下のようなことを考えています。

  • 画像をテーブルにドラッグ・ドロップしたとき
    • ドラッグ・ドロップするとテーブルに行が追加される
    • 複数ファイルをドラッグ・ドロップしたとき、ファイル数に応じた行数が追加される
    • ファイル名がnameに入力された状態で表示される
    • ディレクトリごとドラッグ・ドロップされた場合は上記の挙動を踏襲しつつcategoryにディレクトリ名が入る
  • テーブル操作
    • 範囲選択による一括変更に対応
    • 行番号を選択→削除操作で絵文字登録を抹消可能
    • (余力があったら) 列単位でのフィルタ(Excelとかのアレ)対応
    • name、カテゴリなどの設定はそのままに絵文字画像を差し替える機能
  • その他
    • 1ページに表示する絵文字の数を調節可能
    • 検索ワードで名前、カテゴリ、エイリアス等からフィルタリング可能
    • 確定ボタン(絵文字の登録・更新APIをたたいて一括で反映させる/1回で登録更新できる上限数を設けてもいいかも)
    • リセットボタン(サーバから取り寄せた直後の内容に戻す)

複数件同時に登録・更新に対応する内容ですが、既存のインポート機能とは下記の点から競合しないと考えています。

  • 別サーバからエクスポートしたものをそのままインポートできること(エイリアス設定なども引き継がれるはず)
  • バックグラウンドで動作可能
  • 大量件数の登録が可能

@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

今の状態








@samunohito samunohito linked a pull request Feb 28, 2024 that will close this issue
5 tasks
@samunohito samunohito added the [Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features. label Nov 27, 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
Development

Successfully merging a pull request may close this issue.