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

Make role tag clickable on user pages #11019

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

yuriha-chan
Copy link
Contributor

What

Fixes #11018
ユーザーページ(https://misskey.example.com/@user_name) にある、そのユーザーが所属するロール一覧をロールのページ(https://misskey.example.com/roles/********) へのリンクにする。

Why

一般ユーザーは、自分の所属していないロールのロールタイムラインにきわめてアクセスしずらい。(私はAPIをたたいてロールIDを取得することでURLをみつけている…)
したがって、新規登録用のユーザーのロールのタイムラインを有効にしていても、交流を促す効果が低い。
ユーザーのロールタグをクリック可能にすることで、ユーザーがロールタイムラインの機能の存在を認識しやすくなり、またアクセスしやすくなる。

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jun 15, 2023
@github-actions github-actions bot requested review from syuilo and tamaina June 15, 2023 15:37
@codecov
Copy link

codecov bot commented Jun 15, 2023

Codecov Report

Merging #11019 (18379d4) into develop (f5dfb64) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop   #11019      +/-   ##
===========================================
- Coverage    77.47%   77.44%   -0.04%     
===========================================
  Files          909      907       -2     
  Lines        91516    91359     -157     
  Branches      6890     6891       +1     
===========================================
- Hits         70904    70749     -155     
+ Misses       20612    20610       -2     
Impacted Files Coverage Δ
packages/frontend/src/pages/user/home.vue 92.83% <100.00%> (+0.02%) ⬆️

... and 3 files with indirect coverage changes

@u1-liquid
Copy link
Contributor

FYI: 公開ロールの場合は
「みつける」→上段の「ロール」→ロールを選択→上段の「タイムライン」
からロールタイムラインへのアクセスが可能です

@yuriha-chan
Copy link
Contributor Author

yuriha-chan commented Jun 16, 2023

現状、公開ロール (isPublic) と「みつける」への掲示(IsExplorable)は別設定になっています。
「みつける」への掲示は手動設定ロールのみが対象になっています。

@yuriha-chan
Copy link
Contributor Author

ロールタイムラインが実装される前(13.9.2)に、(メンバー一覧が表示されない)コンディショナルロールが「みつける」で非表示になったからNew User(相当のロール)が表示されないのか

@syuilo syuilo merged commit 33a2c0b into misskey-dev:develop Jun 23, 2023
@syuilo
Copy link
Member

syuilo commented Jun 23, 2023

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
3 participants