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

自動退会のメールに記載されるお問い合わせ先をlokkaに変更した #7259

Merged

Conversation

reckyy
Copy link
Contributor

@reckyy reckyy commented Jan 26, 2024

Issue

概要

休会後、六ヶ月が経過し自動退会した方に送信される、お問い合わせ先を

  • info@fjord.jp

から

  • info@lokka.jp

に変更しました。

変更確認方法

  1. ブランチ feature/update_to_lokka_auto_unsubscribe_email_address をローカルに取り込む
  2. bin/rails db:seed
  3. export TOKEN=hoge を実行
  4. foreman start -f Procfile.dev を実行し、komagataでログイン
    管理ページの休会ユーザー一覧ページ(/admin/users?target=hibernatedに遷移し、そこにnagai-kyuukai(Nagai Kyu kai)ユーザーとkyuukai(Kyu Kai)ユーザーがいるのを確認。また、nagai-kyuukai(Nagai Kyu kai)ユーザーの詳細ページで、休会日が6ヵ月以上前であることを確認する。
スクリーンショット 2024-01-26 12 30 12 スクリーンショット 2024-01-26 12 30 20
  1. Railsコンソールを操作し、kyuukai(Kyu Kai)ユーザーを削除しておく
    ※このユーザーにはダミーのサブスクリプション番号が登録されており、このユーザーを退会させようとすると定期支払い削除処理でエラーになるため
> user = User.find_by(login_name: 'kyuukai')
> user.destroy!
  1. ブラウザで http://localhost:3000/scheduler/daily/auto_retire?token=hoge にアクセス(画面は表示されないが、処理が走る)

  2. 管理ページをリロードし、先ほどのユーザーが休会ユーザー一覧に表示されておらず、退会ユーザー一覧に表示されることを確認した後、 http://localhost:3000/letter_opener でメール内で、お問い合わせ先のアドレスが変わっていることを確認。(Screenshotの変更後の画面と同様のため、メールの画面はここでは省略します。)

スクリーンショット_2024-01-26_20_19_11

Screenshot

変更前

スクリーンショット_2024-01-26_20_23_25

変更後

スクリーンショット_2024-01-26_18_54_53

@reckyy reckyy marked this pull request as ready for review January 26, 2024 11:35
@reckyy reckyy requested a review from ham-cap January 26, 2024 11:36
@reckyy
Copy link
Contributor Author

reckyy commented Jan 26, 2024

@ham-cap
お疲れ様です!
こちらのPRのレビューをお願いしたく、ご連絡いたしました。
お手隙の際にご対応いただけると嬉しいです。
もし、ご都合悪ければ仰ってください!
よろしくお願いいたします。

@reckyy reckyy self-assigned this Jan 26, 2024
@reckyy
Copy link
Contributor Author

reckyy commented Jan 27, 2024

@ham-cap
お疲れ様です!
CIが通っていない状態で、レビュー依頼してしまっていました。
申し訳ございません。
原因のIssue #7255 がマージされるまで待ちたいと思うので、昨日のレビュー依頼はご放念ください。 🙇

@reckyy reckyy removed the request for review from ham-cap January 27, 2024 12:22
@reckyy reckyy force-pushed the feature/update_to_lokka_auto_unsubscribe_email_address branch from a28af2d to b740f52 Compare January 27, 2024 22:52
@reckyy
Copy link
Contributor Author

reckyy commented Jan 28, 2024

@ogawa-tomo

fjord choiceのメンバーにいらっしゃったので、レビュー依頼を送信してしまいました。
下記依頼はご放念いただければ幸いです。

お疲れ様です!
こちらのPRのレビューをお願いしたく、ご連絡いたしました。
お手隙の際にご対応いただけると嬉しいです。
もし、ご都合悪ければ仰ってください!
よろしくお願いいたします。

@reckyy reckyy requested review from niikz and removed request for ogawa-tomo January 28, 2024 02:53
@reckyy
Copy link
Contributor Author

reckyy commented Jan 28, 2024

@niikz
お疲れ様です!
こちらのPRのレビューをお願いしたく、ご連絡いたしました。
お手隙の際にご対応いただけると嬉しいです。
もし、ご都合悪ければ仰ってください!
よろしくお願いいたします。

@niikz
Copy link
Contributor

niikz commented Jan 30, 2024

@reckyy
お疲れさまです。確認が遅くなってしまい申し訳ありません…🙏
1週間以内に対応いたします🙇 よろしくお願いします!

Copy link
Contributor

@niikz niikz left a comment

Choose a reason for hiding this comment

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

@reckyy
確認できました!私からはOKです〜🙆‍♀️
変更確認方法も丁寧に説明されていて分かりやすかったです!
6番目の文章「http://localhost:3000/letter_opener でメール…」の後が途切れているかもと思いました(勘違いだったらすみません🙏)

@reckyy
Copy link
Contributor Author

reckyy commented Feb 2, 2024

@niikz
Approveありがとうございます〜!
文章の漏れのご指摘、ありがとうございます!修正しておきました 🙇

@reckyy
Copy link
Contributor Author

reckyy commented Feb 2, 2024

@komagata
お疲れ様です。
チームメンバーの方にApproveいただいたので、こちらレビューお願いいたします!

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 037a751 into main Feb 2, 2024
7 checks passed
@komagata komagata deleted the feature/update_to_lokka_auto_unsubscribe_email_address branch February 2, 2024 18:39
@github-actions github-actions bot mentioned this pull request Feb 2, 2024
11 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.

None yet

3 participants