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

テスト環境におけるイベント日時の秒数入力フォームを削除 #7441

Merged

Conversation

umizaru
Copy link
Contributor

@umizaru umizaru commented Feb 26, 2024

Issue

概要

テスト環境において、特別イベントを編集するとき画面上に表示される秒数入力の部分を表示されないようにしました。

変更確認方法

  1. bug/remove_seconds_input_form_of_event_date_in_test_environmentをローカルに取り込む
  2. git checkout bug/remove_seconds_input_form_of_event_date_in_test_environmentでローカルブランチを移動
  3. rails db:reset
  4. bin/setupを実行
  5. foreman start -f Procfile.devを実行
  6. 以下4つのページにアクセスし、イベント開始日時イベント終了日時募集開始日時募集終了日時の入力欄で秒数が表示されていないことを確認
    募集期間前のイベント
    募集期間後のイベント
    募集期間中のイベント(補欠者なし)
    募集期間中のイベント(補欠者あり)

Screenshot

変更前

スクリーンショット 2024-02-26 22 15 53

変更後

スクリーンショット 2024-02-26 21 56 49

@umizaru umizaru self-assigned this Feb 26, 2024
@umizaru umizaru requested review from a-terumoto-gs and removed request for a-terumoto-gs February 26, 2024 13:25
@umizaru
Copy link
Contributor Author

umizaru commented Feb 26, 2024

@a-terumoto-gs
お疲れさまです!こちらのPRのレビューをお願いできますでしょうか?
もし難しいようでしたら遠慮なく言ってください!🙏

@a-terumoto-gs
Copy link
Contributor

おつかれさまです!

レビュー依頼ありがとうございます。
急ぎではなかったでしょうか?
今週中には終わらせられるようにしたいと思います!

よろしくお願いいたしますm(__)m

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Feb 28, 2024

@umizaru
お待たせしました!
PRがDraft状態のままなのでReady for reviewを押していただきたいです!

書いていただいている通りの手順で自身の環境で変更を確認したところ、4つの該当ページすべてで秒数表示がされる状態のままとなっておりました。
取り込んだブランチのコードの該当箇所に変更が加えられているのも確認し、bin/setupも行いましたが、うまくいかず…
ユーザーは管理者(komagata)でログインして確認しています
こちらの確認の方法・環境の問題かもしれないのですが、再度確認お願いしてもよろしいでしょうか?
こちらでも調査してみます。
image

また、変更前の画面のスクリーンショットも追加していただけるとわかりやすさが増してより良いと思います!

よろしくお願いしますm(__)m

@a-terumoto-gs
Copy link
Contributor

a-terumoto-gs commented Feb 28, 2024

@umizaru
上記の変更が確認できなかった件について、原因がわかりましたので報告いたします。
結論として、こちらの環境の問題でした。

DBがそのままだったことが変更が反映されない原因だったようで、
rails db:reset実行後、再度確認したところ秒数が表示されないようになりました!

また追加で細かいところですが、変更確認のために貼ってある4つのリンクのうち最後のものだけ編集画面になっていませんでした!
そこも形式合わせていただけるとよりよくなると思います!

コードの変更点には問題ないと思いますので、
Ready for reviewされて、変更前の画面のスクリーンショットの追加が確認出来たらApproveさせて頂きたいと思いますm(__)m
よろしくお願いいたします(^^)/

@umizaru umizaru marked this pull request as ready for review February 28, 2024 12:30
@umizaru
Copy link
Contributor Author

umizaru commented Feb 28, 2024

@a-terumoto-gs
早速レビューいただいてありがとうございました!
また、色々と細かい点が抜けてしまっていてすいません…以下、対応しました!

  • 変更確認方法にrails db:resetを追加
  • 変更前のスクリーンショットを追加
  • PRをReady for Ready状態に変更

全体見ていただいて問題なさそうでしたらApproveいただければと思います!

Copy link
Contributor

@a-terumoto-gs a-terumoto-gs left a comment

Choose a reason for hiding this comment

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

@umizaru
対応ありがとうございます。確認しました!
わたしからはApproveとさせていただきます!
おつかれさまでした(^^)

@umizaru
Copy link
Contributor Author

umizaru commented Apr 15, 2024

@komagata
お疲れさまです!チームメンバーからApproveいただきましたので、コードレビューお願いいたします!

@umizaru umizaru requested a review from komagata April 16, 2024 07:33
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 0d433f1 into main Apr 24, 2024
10 checks passed
@komagata komagata deleted the bug/remove_seconds_input_form_of_event_date_in_test_environment branch April 24, 2024 02:32
@komagata komagata mentioned this pull request Apr 24, 2024
11 tasks
@umizaru
Copy link
Contributor Author

umizaru commented Apr 25, 2024

リリースされたのでクローズします!

@komagata
Copy link
Member

@umizaru リリースされたらクローズではなく、リリース後に本番環境で動作確認をして問題なかったらクローズをお願いしたいです~。

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