Skip to content

フッターにYoutubeチャンネルのリンクを追加#8409

Merged
komagata merged 2 commits intomainfrom
feature/add-youtube-link-to-footer
Mar 22, 2025
Merged

フッターにYoutubeチャンネルのリンクを追加#8409
komagata merged 2 commits intomainfrom
feature/add-youtube-link-to-footer

Conversation

@k-yano-gs
Copy link
Copy Markdown
Contributor

@k-yano-gs k-yano-gs commented Mar 7, 2025

Issue

概要

  • Youtubeチャンネルのリンクをフッターに追加し、クリックすると別タグで開くようにしました

変更確認方法

  1. feature/add-youtube-link-to-footerをローカルに取り込む
  2. foreman start -f Procfile.devでサーバーを立ち上げる
  3. ログインしてページ下部のフッターに「Youtubeチャンネル」が追加されていることを確認する

Screenshot

変更前

image

変更後

image

2025-03-07134931-ezgif com-video-to-gif-converter

@k-yano-gs k-yano-gs self-assigned this Mar 7, 2025
@k-yano-gs k-yano-gs force-pushed the feature/add-youtube-link-to-footer branch 2 times, most recently from efe3130 to 3398359 Compare March 11, 2025 00:36
@k-yano-gs k-yano-gs requested a review from a user March 11, 2025 01:23
@k-yano-gs
Copy link
Copy Markdown
Contributor Author

@harada-webdev
お疲れ様です!
こちらのIssueのレビューをお願いしたいです。よろしくお願いします<( _ _ )>

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

コメントしました!

= link_to 'https://roulette-talk.com/', class: 'footer-nav__item-link', target: '_blank', rel: 'noopener' do
| RouletteTalk
li.footer-nav__item
= link_to 'https://www.youtube.com/@%E3%83%95%E3%82%A3%E3%83%A8%E3%83%AB%E3%83%89%E3%83%96%E3%83%BC%E3%83%88%E3%82%AD%E3%83%A3%E3%83%B3%E3%83%97', class: 'footer-nav__item-link', target: '_blank', rel: 'noopener' do
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

https://www.youtube.com/@フィヨルドブートキャンプのように、日本語エンコードする前のリンクに変更したほうが可読性が上がると思います!

参考: #8308 (comment)

@k-yano-gs k-yano-gs force-pushed the feature/add-youtube-link-to-footer branch from 3398359 to bd7b1d2 Compare March 13, 2025 04:29
@k-yano-gs
Copy link
Copy Markdown
Contributor Author

@harada-webdev
ありがとうございます。
ご指摘して下さったようにエンコードする前のリンクに変更いたしました。
ご確認よろしくお願いいたします<( _ _ )>

Copy link
Copy Markdown

@ghost ghost left a comment

Choose a reason for hiding this comment

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

OKです!
Approveしました!

@k-yano-gs
Copy link
Copy Markdown
Contributor Author

@harada-webdev
ありがとうございます!

@komagata
Approveしていただいたのでレビューをお願い致します<( _ _ )>

@k-yano-gs k-yano-gs requested a review from komagata March 13, 2025 06:05
@komagata
Copy link
Copy Markdown
Member

@kousei-ya conflictの解消をお願いします〜

@k-yano-gs k-yano-gs force-pushed the feature/add-youtube-link-to-footer branch from bd7b1d2 to c05994f Compare March 21, 2025 00:30
@k-yano-gs
Copy link
Copy Markdown
Contributor Author

@komagata
conflictの解消を行おうと思ったのですが、2つ目のコミットが反映されておらずもう一度同じ変更を実施いたしました。最終的にFiles changesは適切な内容になり、動作も問題なかったのですが手順を誤ってしまったかもしれません。
問題ないかご確認お願いいたします<( _ _ )>

@komagata
Copy link
Copy Markdown
Member

@kousei-ya 同じcommitが複数あるので正しい状態に修正してください。

スクリーンショット 2025-03-21 16 10 03

@k-yano-gs k-yano-gs force-pushed the feature/add-youtube-link-to-footer branch from 471595f to c05994f Compare March 21, 2025 07:47
@k-yano-gs k-yano-gs closed this Mar 21, 2025
@k-yano-gs k-yano-gs force-pushed the feature/add-youtube-link-to-footer branch from c05994f to 01227a8 Compare March 21, 2025 07:51
@k-yano-gs
Copy link
Copy Markdown
Contributor Author

k-yano-gs commented Mar 21, 2025

@komagata
こちら最初のコミットの内容が変更されておりその前の状態に戻そうとしたらプルリクがcloseされてしまいましたのでもう一度コミットをやり直しております。

@k-yano-gs k-yano-gs reopened this Mar 21, 2025
Copy link
Copy Markdown
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 455a5d1 into main Mar 22, 2025
2 checks passed
@komagata komagata deleted the feature/add-youtube-link-to-footer branch March 22, 2025 21:49
@github-actions github-actions bot mentioned this pull request Mar 22, 2025
15 tasks
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.

2 participants