Ekke

Results 3 comments of Ekke

OSSに関わる経験が浅いのでコメントの仕方とかが間違ってたらすみません ちょうど同じissueに取り組んでいていくつか試行錯誤していました。 Additional infoで言及されていますが、条件の範囲を逆にした (/^[a-z0-9_]+$/) 等の正規表現の方が良いと思いました。 ### 背景 - 新規絵文字追加の際には、以下のように使用できる文字かどうかで判定がされていること https://github.com/misskey-dev/misskey/blob/63df2c851e31a30dda3b7e9edefb2a97e4c1daa6/packages/backend/src/server/api/endpoints/admin/emoji/add.ts#L51 - unicode絵文字、@等を含む文字列以外でも正しく表示されない場合があること ローカル環境で試したところ、こちらのPRで修正していただいている場合以外の文字列でも正しく表示されない場合がありました。 例 ) カスタム絵文字の名前を試しに「にほんご」に設定すると、👍と表示される ![image](https://user-images.githubusercontent.com/71656810/219394852-32dae93d-e469-4ecd-9553-f3a09694aee6.png)

さっそく変更ありがとうございます。 > ただ、絵文字に使える文字でどこまで許容(どの文字まで使えるように)するかが問題になってきそうですね たしかにその問題は出てきますね... 個人的には、今変更していただいた (/^[a-z0-9_]+$/) で、新規絵文字追加とも同じになって良いと思いました。

I submitted Pull Request about this issue before. - #1832 This PR is not merged yet, but I apply this change to my app by using patch-package (https://www.npmjs.com/package/patch-package) I hope...