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

つながりの公開範囲を非公開に設定しているとき、ユーザーページのフォロー/フォロワー数の表示は「0」ではなく「非公開」とする #10887

Closed
mappi-pr opened this issue May 24, 2023 · 1 comment
Labels
✨Feature This adds/improves/enhances a feature

Comments

@mappi-pr
Copy link
Contributor

Summary

バージョン13.12.1現在、つながりの公開範囲を非公開に設定しているとき、ユーザーページのフォロー/フォロワー数の表示は「0」となりリンクもクリックができ遷移先でエラーのページが表示されるようになっています。
ユーザーページのフォロー/フォロワー数の表示を「非公開」とし、リンクもクリックできないようにすることでUIの改善を図りたいです。
※Misskey.ioで0/0表示に疑問を持っている声をいくつか見ております。

想定される経緯

#8032 (comment)

(フォロー・フォロワーの数は0として表記するほうがよさそう?)

⇒ユーザーページ参照可能とした際に、フォロー・フォロワーの数は0となったのではないかと思われます。

対応イメージ

image

課題

①フォロー・フォロワー数を表示している箇所が多そう

影響範囲を確認しきれていないので、
misskey.entities.UserDetailed.followersCount
misskey.entities.UserDetailed.followingCount
の利用箇所に横展開が必要?コメントいただけましたら幸いです。

②表示パターンの網羅と、自動テストへの影響

自分自身の情報を参照するとき・「フォロワーのみ」設定の場合の処理の考慮が必要そう。
(テストケースの追加が必要?)
ロジック追加しすぎてパフォーマンスに影響があるようならば取り下げにしようかと思います。

@mappi-pr mappi-pr added the ✨Feature This adds/improves/enhances a feature label May 24, 2023
@mappi-pr
Copy link
Contributor Author

フォロー・フォロワー数を表示している箇所が多そう

この辺が網羅できていればいいのかしら・・・?
image
image

自分自身の情報を参照するとき・「フォロワーのみ」設定の場合の処理の考慮が必要そう。

これはいけそう
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

No branches or pull requests

1 participant