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に更新する #7337

Conversation

a-terumoto-gs
Copy link
Contributor

@a-terumoto-gs a-terumoto-gs commented Feb 8, 2024

Issue

概要

「リンク切れを調べてメールを出す」機能の、メール宛先がinfo@fjord.jpになっているものをinfo@lokka.jpに更新する

変更確認方法

  1. feature/update-check-broken-link-function-email-destination-to-lokkaをローカルに取り込む
  2. 該当の2ファイルpp/mailers/check_url_mailer.rbtest/mailers/check_url_mailer_test.rbの送信先アドレス箇所を確認する
    ※現在動いていない機能なので動作確認は不要です

Screenshot

画面のない機能であるため、コードの該当箇所のスクリーンショットです。

変更前

メールアドレス変更前 対象コード

メールアドレス変更前 テストコード

変更後

メールアドレス変更後 対象コード

スクショ メールアドレス変更後 テストコード

@a-terumoto-gs a-terumoto-gs self-assigned this Feb 8, 2024
@a-terumoto-gs a-terumoto-gs marked this pull request as ready for review February 8, 2024 06:00
@a-terumoto-gs a-terumoto-gs requested a review from niikz February 8, 2024 06:00
@a-terumoto-gs
Copy link
Contributor Author

@niikz
お疲れ様です!
お手すきの際にこちらのレビューをお願いしてもよろしいでしょうか?
よろしくお願いいたします。

@niikz
Copy link
Contributor

niikz commented Feb 10, 2024

@a-terumoto-gs
確認できました!私からはOKです〜🙆‍♀️

@a-terumoto-gs
Copy link
Contributor Author

@niikz ありがとうございますm(__)m

@a-terumoto-gs
Copy link
Contributor Author

@komagata
お疲れ様です。
メンバーの方からOKいただいたので、
お手すきの際にこちらのレビューをお願いしてもよろしいでしょうか?
よろしくお願いいたします。

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 e24e51a into main Feb 15, 2024
12 checks passed
@komagata komagata deleted the feature/update-check-broken-link-function-email-destination-to-lokka branch February 15, 2024 10:17
@github-actions github-actions bot mentioned this pull request Feb 15, 2024
19 tasks
@a-terumoto-gs
Copy link
Contributor Author

メモ
動作確認は必要ないのでリリースブランチにチェック済み

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