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

イベント一覧に定期イベント作成リンクを追加 #5753

Merged
merged 2 commits into from
Nov 18, 2022

Conversation

sadanora
Copy link
Contributor

@sadanora sadanora commented Nov 9, 2022

Issue

概要

イベント一覧に定期イベント作成リンクを追加しました。

変更確認方法

  1. ブランチfeature/add-creating-regular-event-link-to-eventsをローカルに取り込んでください。
  2. bin/rails sでローカル環境を立ち上げてください。
  3. http://localhost:3000 にアクセスし、任意のユーザーでログインしてください。
  4. 左側メニューの「イベント」リンクをクリックしイベント一覧を表示してください。
  5. 右上に「+定期イベント作成」のリンクがあることを確認し、クリックすると定期イベント作成画面に遷移することを確認してください。

変更前

before

変更後

after

@sadanora sadanora self-assigned this Nov 9, 2022
@sadanora sadanora changed the title イベント一覧に定期イベント作成リンクが欲しい #5693 イベント一覧に定期イベント作成リンクが欲しい Nov 9, 2022
@sadanora sadanora changed the title イベント一覧に定期イベント作成リンクが欲しい イベント一覧に定期イベント作成リンクを追加 Nov 9, 2022
@sadanora sadanora marked this pull request as ready for review November 9, 2022 08:14
@sadanora
Copy link
Contributor Author

sadanora commented Nov 9, 2022

@hikarook94
おつかれさまです!
こちらのレビューをお願いしたいのですが、ご都合いかがでしょうか?:pray:
もし厳しければ他の方にお願いしますので仰ってください:bow:

@hikarook94
Copy link
Contributor

@sadanora
お疲れ様です!返信遅くなりすみません🙏
確認させていただきます👍

@sadanora
Copy link
Contributor Author

@hikarook94
お忙しい中ありがとうございます!
急ぎませんのでご都合の良い時にお願いできたらと思います!
よろしくおねがいいたします〜!:pray:

Copy link
Contributor

@hikarook94 hikarook94 left a comment

Choose a reason for hiding this comment

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

@sadanora
お疲れ様です!動作確認、コードともに問題ないと思います!
Approveさせていただきます!

@sadanora
Copy link
Contributor Author

@hikarook94
ご確認ありがとうございます!
早速のレビューありがとうございました!

@sadanora
Copy link
Contributor Author

@komagata
お疲れ様です!
メンバーレビューでapproveをいただいたので、レビューをお願いいたします!

@@ -8,6 +8,11 @@ class EventsTest < ApplicationSystemTestCase
assert_link 'イベント作成'
end

test 'show link to create new regular event' do
Copy link
Member

Choose a reason for hiding this comment

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

要素を追加したぐらいの内容であればsystem testはなしで大丈夫です。
理由は下記と同じになります〜

#5756 (comment)

@komagata
Copy link
Member

📝 @sadanora さんからMTG中に下記の質問があった

「自分の作ったテスト以外にも、リンクの存在を確かめるテストがあるが、そちらも修正した方がいいか。」

下記それに対する返答。

「もし修正してもらえるのであれば大変ありがたいです〜!」

@sadanora sadanora force-pushed the feature/add-creating-regular-event-link-to-events branch from f5ac74b to 901dc37 Compare November 16, 2022 06:48
@sadanora
Copy link
Contributor Author

@komagata

「もし修正してもらえるのであれば大変ありがたいです〜!」

回答ありがとうございます!
自分の書いていないテストも含めて削除しました。再レビューよろしくお願いいたします!

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 8badf6d into main Nov 18, 2022
@komagata komagata deleted the feature/add-creating-regular-event-link-to-events branch November 18, 2022 05:34
@github-actions github-actions bot mentioned this pull request Nov 18, 2022
18 tasks
@sadanora
Copy link
Contributor Author

@komagata
レビューありがとうございました〜!

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