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

トップページのRubyKaigi2022横断幕バナーを削除 #5812

Merged
merged 2 commits into from
Nov 21, 2022

Conversation

peno022
Copy link
Contributor

@peno022 peno022 commented Nov 19, 2022

Issue

概要

RubyKaigi2022は終了しているため、トップページのRubyKaigi2022横断幕バナーを削除しました。
あわせて、削除されていなかったRubyKaigi2021のバナー画像も削除しました。

...

変更確認方法

  1. ブランチfeature/remove-RubyKaigi2022-banner-from-top-pageをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. http://127.0.0.1:3000/ にアクセスする

変更前

202211180903_変更前トップ画面_切り取り

変更後

変更後screencapture-127-0-0-1-3000-2022-11-19-09_06_06_切り抜き

確認したいこと

@peno022 peno022 self-assigned this Nov 19, 2022
@machida
Copy link
Member

machida commented Nov 19, 2022

@mayumonj 確認ありがとうございます!消し忘れの方も削除お願いしたいですー🙏

@peno022 peno022 marked this pull request as ready for review November 19, 2022 23:34
@peno022 peno022 requested a review from shucream-p November 20, 2022 01:14
@peno022
Copy link
Contributor Author

peno022 commented Nov 20, 2022

@shucream-p
おつかれさまです!ご都合よろしければ、このPRのレビューをお願いできないでしょうか?
私がチーム開発に参加して初めて担当するissueなので、基本的なことの不備等もあるかもしれません。。なんでもご指摘いただけますと幸いです🙏

Copy link
Contributor

@shucream-p shucream-p left a comment

Choose a reason for hiding this comment

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

お疲れ様です!
返信遅くなりすみません🙇‍♂️
こちら動作とコード確認しました。問題ないと思いますのでApproveさせていただきます!

@peno022
Copy link
Contributor Author

peno022 commented Nov 21, 2022

@shucream-p ありがとうございますー!🙌✨

@peno022 peno022 requested a review from komagata November 21, 2022 06:18
@peno022
Copy link
Contributor Author

peno022 commented Nov 21, 2022

@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 816b8d0 into main Nov 21, 2022
@komagata komagata deleted the feature/remove-RubyKaigi2022-banner-from-top-page branch November 21, 2022 13:29
@github-actions github-actions bot mentioned this pull request Nov 21, 2022
18 tasks
@peno022
Copy link
Contributor Author

peno022 commented Nov 21, 2022

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

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.

4 participants