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

ブログ記事作成時に通知が飛ばないよう、該当部分のコードをコメントアウト #8281

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ayu-0505
Copy link
Contributor

@ayu-0505 ayu-0505 commented Jan 20, 2025

Issue

概要

ブログ記事を作成した時に通知が飛ばないように該当部分をコメントアウトしました。
この変更は #8244 の変更作業のための一時的な処理です。

変更確認方法

  1. {chore/pause-article-submission-notifications}をローカルに取り込む
    1. git fetch origin pull/8281/head:chore/pause-article-submission-notifications を取り込む (2度目以降は--forceをつけてください)
    2. git switch chore/pause-article-submission-notificationsでブランチ移動
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる。
  3. 適当なメンター(例: user:komagata, password: testtest)でログインする。
  4. ブログ記事作成ページにおいて、適当にタイトルと本文を入力し、公開するボタンをクリックする。
  5. ブログ記事一覧に移動し、作成した記事が公開されているか確認する。
  6. 現在のユーザーからログアウトし、適当な他ユーザーでログインする。(例:user: machida, password: testtest)
    通知が飛んでいるかどうかの確認なので、ユーザーのロール(メンターか受講生か、等)は問いません。
  7. ページ右上の通知で、先ほど公開したブログ記事の通知が飛んでいないことを確認する。

Screenshot

変更前

  • 記事作成時に以下のような通知した旨の表示が別タブで出ます。
スクリーンショット 2025-01-23 17 07 06
  • 他ユーザーでログインすると通知が飛んでいることが確認できます。
スクリーンショット 2025-01-23 17 08 03

変更後

  • 変更前と同様に記事を作成しますが、公開するボタンを押してもそのまま公開記事に移動します。
    記事一覧に行くとちゃんと公開されていることが確認できます。
スクリーンショット 2025-01-23 17 12 19 スクリーンショット 2025-01-23 17 12 45
  • 他ユーザーでログインすると通知が飛んでいないことが確認できます。
スクリーンショット 2025-01-23 17 13 21

@ayu-0505 ayu-0505 self-assigned this Jan 20, 2025
@mousu-a mousu-a marked this pull request as ready for review January 22, 2025 02:42
@mousu-a mousu-a marked this pull request as draft January 22, 2025 02:42
@mousu-a
Copy link
Contributor

mousu-a commented Jan 22, 2025

@ayu-0505
すみません、こちらのPRを操作をミスしてDraft→Openにしてしまいました🙇‍♂️(すでに戻してあります)
(自分のPRのCIが中々通らず、他の人のところでも同じ箇所で落ちていないか確認していたところの操作ミスでした)
お騒がせしてしまい申し訳ないです🙇‍♂️

@ayu-0505
Copy link
Contributor Author

@mousu-a
連絡ありがとうございます!
全然問題ないですよ〜修正いただきありがとうございました🙏

@ayu-0505 ayu-0505 force-pushed the chore/pause-article-submission-notifications branch from be4a491 to 37f5f9e Compare January 24, 2025 00:50
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