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

人気投票関連のバナーを追加しました #377

Merged

Conversation

KiKiKi-KiKi
Copy link
Contributor

@KiKiKi-KiKi KiKiKi-KiKi commented Feb 18, 2020

変更内容: Summary

人気投票の動画の下に おめシスの人気キャラクター投票, おめシスの人気動画投票 のバナーを追加しました〜
該当動画の直下に投票フォームのバナーがあることでサイトで動画を見てそのままアクセスしやすくなるのではないかと思います。

📱
banners mb

💻
banners pc

※ title, alt のテキストは YouTube動画の概要から、バナー画像の元データは投票フォームのheaderから取っています

確認事項: Check point

  • PR を作成する前に、 https://github.com/omegasisters/homepage の最新の master を取り込み済みである。
    • Conflict や他の方の変更で自分の変更が動かなくなる可能性を防ぎます。
    • 最新の master を取り込む方法
      • upstream に fork 元リポジトリを追加
      • 現在のブランチに upstreammaster を取り込む
        • $ git pull --rebase upstream master
    • おまけ
      • rebase 後に再度 push する場合、 --force-with-lease オプションをつける
        • git push --force-with-lease origin <ブランチ名>
  • 動作確認済みである。
    • 何らかの理由で本番に取り込まれるまで確認できない場合はその旨を補足に記載する。
  • prettier によるコード整形を行った、もしくは画面に関係ない変更である。
    • 可能な方のみで良いと思いますが、意図せず他の方がフォーマットするとコード差分が増えすぎるので自分の分は自分でやるのがよろしいかと思います。
  • スマホ(狭い画角)でも表示を確認した、もしくは画面に関係ない変更である。
  • 他の方の変更を意図せず削除・変更していないか、差分をもう一度確認した。
  • 破壊的な変更を行った場合、影響範囲をもう一度確認した。もしくは破壊的な変更を行っていない。
  • Pull Request に関連した issue の URL を貼り付けた

補足: Other Information

#369

@KiKiKi-KiKi KiKiKi-KiKi changed the title 人気投票関連のバナーを追加しました [WIP]人気投票関連のバナーを追加しました Feb 18, 2020
@KiKiKi-KiKi KiKiKi-KiKi force-pushed the popularity_vote_campaign_banners branch from 260b76a to 6132a9b Compare February 18, 2020 16:17
@KiKiKi-KiKi KiKiKi-KiKi changed the title [WIP]人気投票関連のバナーを追加しました 人気投票関連のバナーを追加しました Feb 18, 2020
@omegasisters omegasisters merged commit b028021 into omegasisters:master Feb 20, 2020
@omegasisters
Copy link
Owner

👍

@KiKiKi-KiKi KiKiKi-KiKi deleted the popularity_vote_campaign_banners branch February 20, 2020 22:59
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