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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 3 commits into from
Nov 15, 2022
Merged

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

merged 3 commits into from
Nov 15, 2022

Conversation

sadanora
Copy link
Contributor

@sadanora sadanora commented Nov 9, 2022

Issue

概要

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

変更確認方法

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

変更前

before

変更後

after

@sadanora sadanora self-assigned this 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

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

@fuwa-syugyo
Copy link
Contributor

@sadanora
お疲れ様です〜！
レビュー依頼ありがとうございます🙏
後ほど対応させていただきます〜！

@sadanora
Copy link
Contributor Author

sadanora commented Nov 9, 2022

@fuwa-syugyo
わー、お忙しい中ありがとうございます〜！
急ぎませんのでお手隙の際にお願いできたらと思います:bow:
よろしくお願いいたします！

@fuwa-syugyo
Copy link
Contributor

@sadanora
お疲れ様です！
定期イベント選択時に「+イベント作成」ボタンが表示されていることを確認しました〜
問題ないと思いますのでApproveさせていただきます！
image

@sadanora
Copy link
Contributor Author

@fuwa-syugyo
早速のレビュー、ありがとうございました！

@sadanora
Copy link
Contributor Author

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

komagata
komagata reviewed Nov 11, 2022
View reviewed changes
test/system/regular_events_test.rb Outdated
Comment on lines 6 to 14
test 'show link to create new event' do
visit_with_auth regular_events_path, 'komagata'
assert_link 'イベント作成'
end

test 'show link to create new regular event' do
visit_with_auth regular_events_path, 'komagata'
assert_link '定期イベント作成'
end
Copy link
Member

Choose a reason for hiding this comment

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

この程度であればテストを作成しなくて大丈夫です〜。

systemテストはすごく遅いので、重要なものだけに絞りたいです。もしやるとしてもviewのテストとしてやりたい感じです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

承知しました！
追加したテストケースは削除しました〜
9f9f3b9

@sadanora sadanora force-pushed the feature/add-creating-event-link-to-regular_events branch from 5cf302b to 9f9f3b9 Compare November 11, 2022 07:58
komagata
komagata reviewed Nov 14, 2022
View reviewed changes
test/system/regular_events_test.rb Outdated
@@ -3,7 +3,7 @@
require 'application_system_test_case'

class RegularEventsTest < ApplicationSystemTestCase
test 'create regular event as WIP' do
test 'show regular event as WIP' do
Copy link
Member

Choose a reason for hiding this comment

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

@sadanora こちらを変更されたのは何故でしょうか？

Copy link
Contributor Author

@sadanora sadanora Nov 15, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

@komagata
変更した理由に覚えがないので、テストケースを追加した際に誤って変更してしまったんだと思います。
大変失礼いたしました:bow:

意図しない変更だったので元に戻し、このPRのFile Changedタブでテストファイルに差分が出ていないことを確認しました。

@sadanora sadanora force-pushed the feature/add-creating-event-link-to-regular_events branch from 9f9f3b9 to 5722d49 Compare November 15, 2022 00:23
komagata
komagata approved these changes Nov 15, 2022
View reviewed changes
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 740dd07 into main Nov 15, 2022
@komagata komagata deleted the feature/add-creating-event-link-to-regular_events branch November 15, 2022 04:22
@github-actions github-actions bot mentioned this pull request Nov 15, 2022
Merged
19 tasks
@komagata komagata mentioned this pull request Nov 12, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fuwa-syugyo fuwa-syugyo fuwa-syugyo approved these changes

@komagata komagata komagata approved these changes

Assignees

@sadanora sadanora

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sadanora @fuwa-syugyo @komagata