search

misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
9.88k stars 1.33k forks source link

Assign role to remote user on user management page #14343

Open nakkaa opened 1 month ago

nakkaa commented 1 month ago

💡 Summary

リモートユーザーにロールをアサインできません。

🥰 Expected Behavior

アサインボタンが表示される。

🤬 Actual Behavior

アサインボタンが表示されない。

📝 Steps to Reproduce

  1. 管理者アカウントでリモートユーザーのプロフィールを開く
  2. フォローボタン左にある「...」からモデレーションを押下する
  3. 「ロール」タブを押下する。

💻 Frontend Environment

* Model and OS of the device(s): Mac
* Browser: Safari
* Server URL: https://misskey.7ka.org
* Misskey: 2024.7.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: Docker
* Misskey: 2024.7.0
* Node: -
* PostgreSQL: 16.3
* Redis: 7.x
* OS and Architecture: Ubuntu 22.04

Do you want to address this bug yourself?

nakkaa commented 1 month ago

2024.5.0のサーバーでも再現したので、2024.7.0特有ではなさそうでした。

samunohito commented 1 month ago

以下のような形で明示的に制御されているので、仕様の可能性もありそうです。

https://github.com/misskey-dev/misskey/blob/6dd2e9fc0b1eeea6b5f04ccac93ccfab658f976d/packages/frontend/src/pages/admin-user.vue#L135

Sayamame-beans commented 1 month ago

じゃあfeatreqに変えます…? 変えました リモートユーザーにロールを付けることはある(サイレンス等)ので、ここだけ出来なくする理由はあまり無い気がしています

kakkokari-gtyih commented 1 month ago

たぶん仕様じゃないですかね・・・

https://github.com/misskey-dev/misskey/blob/develop/packages/backend/src/core/entities/UserEntityService.ts#L503-L512

samunohito commented 1 month ago

たぶん仕様じゃないですかね・・・

これは取得時かつバッジの話なので、本件とはちょっと違いそうです （admin-user.vueで見ているユーザがuser.host == nullかどうかなので）

kakkokari-gtyih commented 1 month ago

a-

samunohito commented 1 month ago
  1. 管理者アカウントでリモートユーザーのプロフィールを開く
  2. フォローボタン左にある「...」からロール→任意のロールを押下

でリモートユーザに対してもロールの付与は出来ました。 …ので、冒頭の個所であえてローカルオンリーに制御する必要は無いのかも？