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

enhance: “つながりの公開範囲”がフォロー・フォロワー個別設定できるように #12702

Merged
merged 5 commits into from
Dec 18, 2023

Conversation

zawa-ch
Copy link
Contributor

@zawa-ch zawa-ch commented Dec 17, 2023

What

このPRは「つながりの公開範囲」をフォロー/フォロワーで個別に設定できるようになります。

Why

このPRは #12072 を解決することができます。 (Resolve #12072)

Additional info (optional)

Screenshot 1
Screenshot 2

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 packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR packages/misskey-js packages/backend:test labels Dec 17, 2023
Copy link

codecov bot commented Dec 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (67824cf) 79.65% compared to head (f265ab4) 81.74%.
Report is 10 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12702      +/-   ##
===========================================
+ Coverage    79.65%   81.74%   +2.09%     
===========================================
  Files          955      178     -777     
  Lines       107713    27337   -80376     
  Branches      8351      511    -7840     
===========================================
- Hits         85796    22347   -63449     
+ Misses       21917     4990   -16927     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zawa-ch zawa-ch marked this pull request as ready for review December 17, 2023 19:05
@zawa-ch zawa-ch changed the title Enhance: “つながりの公開範囲”がフォロー・フォロワー個別設定できるように (#12072) enhance: “つながりの公開範囲”がフォロー・フォロワー個別設定できるように Dec 17, 2023
@syuilo
Copy link
Member

syuilo commented Dec 18, 2023

  • ja-JP以外の翻訳ファイルの編集はcrowdinとコンフリクトしてしまうので避けていただけると🙏
  • followerVisibility -> followersVisibility かも

@zawa-ch
Copy link
Contributor Author

zawa-ch commented Dec 18, 2023

  • ja-JP以外の翻訳ファイルの編集はcrowdinとコンフリクトしてしまうので避けていただけると🙏

わかりました、この部分は差し戻します

  • followerVisibility -> followersVisibility かも

ア!

@zawa-ch
Copy link
Contributor Author

zawa-ch commented Dec 18, 2023

  • ja-JP以外の翻訳ファイルの編集はcrowdinとコンフリクトしてしまうので避けていただけると🙏

わかりました、この部分は差し戻します

  • followerVisibility -> followersVisibility かも

ア!

修正しました

ffVisibility: "つながりの公開範囲"
ffVisibilityDescription: "自分のフォロー/フォロワー情報の公開範囲を設定できます。"
followingVisibility: "フォローの公開範囲"
followingVisibilityDescription: "自分のフォロー情報の公開範囲を設定できます。"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

「つながり」から「フォロー/フォロワー」と具体的な呼称に変わったから特にdescriptionは要らなそう

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

そうですね、消しちゃいましょう

followingVisibility: "フォローの公開範囲"
followingVisibilityDescription: "自分のフォロー情報の公開範囲を設定できます。"
followersVisibility: "フォロワーの公開範囲"
followersVisibilityDescription: "自分のフォロワー情報の公開範囲を設定できます。"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

同上

名称が具体的になって必要なくなったため
misskey-dev#12702 (comment)
@zawa-ch
Copy link
Contributor Author

zawa-ch commented Dec 18, 2023

Screenshot

@syuilo syuilo merged commit 4e2d802 into misskey-dev:develop Dec 18, 2023
15 of 18 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 18, 2023

👍👍

camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* Enhance: “つながりの公開範囲”がフォロー・フォロワー個別設定できるように (misskey-dev#12072)

* refactor: crowdin 編集部分のコミットを打ち消し

misskey-dev#12702 (comment)

* refactor: オブジェクトの名前修正

misskey-dev#12702 (comment)

* fix: 設定項目の説明を削除

名称が具体的になって必要なくなったため
misskey-dev#12702 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend:test packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR packages/misskey-js
Projects
None yet
Development

Successfully merging this pull request may close these issues.

“つながりの公開範囲”がフォロー・フォロワー個別設定できるように
2 participants