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

feat: update songs list #438

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vsanna
Copy link
Contributor

@vsanna vsanna commented Jan 28, 2022

変更内容: Summary

  • songsList.jsonを更新しました
    • 新しいカバーが発表されたため
    • 差分ですが、新しいカバーの追加のみならず、全体的に曲順がかわっています。おそらくおめしすサイドで曲順を変更したのかもしれません

したこと。

export YOUTUBE_API_KEY='...........'
yarn genSongsList

確認事項: Check point

  • PR を作成する前に、 https://github.com/omegasisters/homepage の最新の master を取り込み済みである。
  • 動作確認済みである。
  • prettier によるコード整形を行った、もしくは画面に関係ない変更である。
  • スマホ(狭い画角)でも表示を確認した、もしくは画面に関係ない変更である。
  • 他の方の変更を意図せず削除・変更していないか、差分をもう一度確認した。
  • 破壊的な変更を行った場合、影響範囲をもう一度確認した。もしくは破壊的な変更を行っていない。
  • Pull Request に関連した issue の URL を貼り付けた

補足: Other Information

現在、曲順のロジックは以下のとおりです

  1. 常に synchronicity が先頭
  2. その後にYouTube APIの返す position fieldの値順

他の並び順のほうが良ければ変更する余地があります

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.

None yet

1 participant