fix: 非承認制チャンネルでもフォロワーを管理可能にする - #24
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughチャンネル読み込み後のタブ選択を補正し、承認が必要な場合だけフォローリクエストタブを表示します。承認不要チャンネルでは、フォローとフォロワー削除後の状態をE2Eテストで検証します。 Changesチャンネルフォロー承認
Estimated code review effort: 2 (Simple) | ~10 minutes Poem
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
packages/frontend/src/pages/channel.vueParsing error: Unexpected token { Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
このPRによるapi.jsonの差分 |
|
Thank you for sending us a great Pull Request! 👍 example: pnpm run build-misskey-js-with-types |
⚙️ Backend Diagnostics ReportMemory: After GC(No significant changes) V8 Heap Snapshot Statistics
|
What
Why
フォロー承認制ではない既存チャンネルでも、所有者や共同管理者が現在のフォロワーを整理できることを、UIとテストの両方で明確に保証するためです。
Additional info (optional)
channels/followersとchannels/followers/removeを利用します。channel-follow-approval.tsの対象E2Eは2件PASSしました。drop-and-fusion.game.vueにおけるmisskey-bubble-game解決エラーで失敗します。今回の変更箇所に型エラーは出ていません。developに既存のマージ済みmigration 2件のヘッダー欠落によりFAILします。今回の差分にはSPDX対象の新規ファイルはありません。Checklist