Skip to content

Feat/参加中チャンネルの表示#49

Merged
saka-naname merged 5 commits intomainfrom
feat/display-joined-channels
Apr 23, 2025
Merged

Feat/参加中チャンネルの表示#49
saka-naname merged 5 commits intomainfrom
feat/display-joined-channels

Conversation

@saka-naname
Copy link
Contributor

関連 Issue(s)

変更内容

  • 参加中チャンネルを取得・サイドメニューに表示する機能を実装
    • 既存のチャンネル参加処理に、処理後メニュー上のチャンネルリストも更新する処理を追加
  • 参加中チャンネルの表示順・カテゴリを保持できるようスキーマを変更
    • 将来的に並び替えやフォルダ分けが実装できる予定

スクリーンショット

image

確認手順

  1. マイグレーションを更新する (要リセット)
  2. ログイン後、適当なチャンネルページを開く
  3. フッターの「参加する」ボタンを押す
  4. 左のメニュー上に参加したチャンネルが表示される

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 11 out of 13 changed files in this pull request and generated 1 comment.

Files not reviewed (2)
  • prisma/migrations/20250412180407_add_channel_category_and_display_order/migration.sql: Language not supported
  • prisma/schema.prisma: Language not supported

チャンネル参加時はまずデフォルトカテゴリに入る想定、変更を適用

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Member

@newt239 newt239 left a comment

Choose a reason for hiding this comment

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

めちゃめちゃ遅くなってすみません!特段問題は無さそうだったのでLGTMです(チャンネル参加、一覧更新、切り替えも正常に動作することを確認できました)

@saka-naname saka-naname merged commit 8439f26 into main Apr 23, 2025
1 check passed
@saka-naname saka-naname deleted the feat/display-joined-channels branch May 25, 2025 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants