Skip to content

fix: 非承認制チャンネルでもフォロワーを管理可能にする - #24

Merged
mattyatea merged 1 commit into
developfrom
codex/manage-channel-followers
Aug 21, 2026
Merged

fix: 非承認制チャンネルでもフォロワーを管理可能にする#24
mattyatea merged 1 commit into
developfrom
codex/manage-channel-followers

Conversation

@mattyatea

Copy link
Copy Markdown

What

  • チャンネル管理者には、フォロー承認制の有無にかかわらず「フォロワー管理」タブを表示します。
  • 「フォロー申請」タブは、フォロー承認制が有効なチャンネルだけに表示します。
  • チャンネル切り替え後に選択中のタブが利用できない場合は、概要タブへ戻すようにしました。
  • 承認制ではないチャンネルでも、既存フォロワーの一覧取得・削除・件数更新ができるE2Eテストを追加しました。

Why

フォロー承認制ではない既存チャンネルでも、所有者や共同管理者が現在のフォロワーを整理できることを、UIとテストの両方で明確に保証するためです。

Additional info (optional)

  • APIやDBスキーマの変更はありません。既存の channels/followerschannels/followers/remove を利用します。
  • channel-follow-approval.ts の対象E2Eは2件PASSしました。
  • 対象VueのESLintとlocale safetyはPASSしました。
  • frontend全体の型検査は、既存の drop-and-fusion.game.vue における misskey-bubble-game 解決エラーで失敗します。今回の変更箇所に型エラーは出ていません。
  • SPDX全体検査は、developに既存のマージ済みmigration 2件のヘッダー欠落によりFAILします。今回の差分にはSPDX対象の新規ファイルはありません。

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

@coderabbitai

coderabbitai Bot commented Aug 21, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a3ee0441-5b22-49e8-b04d-8e2b41ff51f5

📥 Commits

Reviewing files that changed from the base of the PR and between ff9d84c and 133ee2c.

📒 Files selected for processing (2)
  • packages/backend/test/e2e/channel-follow-approval.ts
  • packages/frontend/src/pages/channel.vue

📝 Walkthrough

Walkthrough

チャンネル読み込み後のタブ選択を補正し、承認が必要な場合だけフォローリクエストタブを表示します。承認不要チャンネルでは、フォローとフォロワー削除後の状態をE2Eテストで検証します。

Changes

チャンネルフォロー承認

Layer / File(s) Summary
チャンネルタブ表示の更新
packages/frontend/src/pages/channel.vue
利用可能なタブに現在のタブがない場合、overviewへ戻します。isFollowApprovalRequired が有効な場合だけ、管理者に「フォローリクエスト」タブを表示します。
フォロワー削除のE2E検証
packages/backend/test/e2e/channel-follow-approval.ts
承認不要チャンネルでのフォロー、フォロワー一覧からの削除、削除後のフォロー状態とフォロワー数を検証します。

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

タブは正しく戻り
承認の門だけ開きます
フォローがつながり
削除で状態がほどけ
テストが静かに確かめます

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/manage-channel-followers

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

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

packages/frontend/src/pages/channel.vue

Parsing 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mattyatea
mattyatea merged commit 97a7973 into develop Aug 21, 2026
20 of 26 checks passed
@github-actions

Copy link
Copy Markdown

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

@github-actions

Copy link
Copy Markdown

Thank you for sending us a great Pull Request! 👍
Please regenerate misskey-js type definitions! 🙏

example:

pnpm run build-misskey-js-with-types

@github-actions

Copy link
Copy Markdown

⚙️ Backend Diagnostics Report

Memory: After GC

(No significant changes)

V8 Heap Snapshot Statistics

Metric @ Base @ Head Δ MAD
$\color{gray}{\rule{8pt}{8pt}}$ Total 89 MB
± 632 B
89 MB
± 11 KB
$\text{-2.1 KB}$
$\text{-0\%}$
11 KB
$\color{orange}{\rule{8pt}{8pt}}$ Code 25 MB 25 MB $\text{-1 KB}$ 12 KB
$\color{red}{\rule{8pt}{8pt}}$ Strings 31 MB 31 MB $\text{-1.1 KB}$ 1.3 KB
$\color{cyan}{\rule{8pt}{8pt}}$ JS arrays 4.7 MB 4.7 MB 0 B 0 B
$\color{green}{\rule{8pt}{8pt}}$ Typed arrays 566 KB 566 KB 0 B 0 B
$\color{yellow}{\rule{8pt}{8pt}}$ System objects 2.3 MB 2.3 MB $\text{+16 B}$ 732 B
$\color{violet}{\rule{8pt}{8pt}}$ Other JS objs 26 MB 26 MB $\text{-288 B}$ 177 B
$\color{pink}{\rule{8pt}{8pt}}$ Other non-JS objs 39 KB 39 KB 0 B 0 B

Download representative heap snapshot: base / head

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant