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

nagai-kyuukaiユーザに分報チャンネルを紐づける #7653

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

ogawa-tomo
Copy link
Contributor

@ogawa-tomo ogawa-tomo commented Apr 10, 2024

Issue

概要

こちらのPRのステージング動作確認用にseedデータを追加するだけのPRです。

このPRの動作確認をするためには、以下の条件を満たすユーザーを作成する必要があります。
・分報チャンネルが存在する
・休会日が六ヶ月以上前である
・Discordアカウント名が不正な値である

そのため、もともと休会日が六ヶ月以上前であるnagai-kyuukaiユーザにステージング環境Discordサーバに存在するチャンネルを紐づけ、さらにDiscordアカウント名を不正にします。
ステージング動作確認に成功すると当然ながらこの分報チャンネルは消えるので、この1回の動作確認のためだけのPRとなります。

変更確認方法

  1. chore/make-times-channel-for-nagai-kyuukai-userをローカルに取り込む
  2. komagataでログイン
  3. 休会中のユーザー一覧から、「nagai-kyuukai」のプロフィールページに遷移し、分報が紐づいていることを確認する

image
4. 分報のリンクをクリックすると、ステージング環境Discordサーバ(fjordbootcamp-staging)の「jidoutaikai20240402」チャンネルに遷移することを確認する

image

@ogawa-tomo
Copy link
Contributor Author

@komagata
下記コメントを受けまして、PRの動作確認のためにseedでデータを用意するためのPRとなります。レビューをお願いできますでしょうか。seedデータ追加のみのPRであること、動作確認にはステージング環境Discordサーバのアクセス権が必要であること(おそらく管理者から招待する必要がある?)から、直接駒形さんにレビュー依頼をしています。
#7515 (comment)

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させていただきました。OKです~👌

@komagata komagata merged commit f806b90 into main Apr 10, 2024
4 checks passed
@komagata komagata deleted the chore/make-times-channel-for-nagai-kyuukai-user branch April 10, 2024 20:34
@github-actions github-actions bot mentioned this pull request Apr 10, 2024
15 tasks
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.

2 participants