onsd commented 3 years ago

WHY

51

プレイリストを作成する関数とTypesをいじった

export const saveSpotifyPlaylist = functions.https.onCall(addSpotifyPlaylistHandler); を作った
- https://github.com/kanko-dori/favox/pull/94/files#diff-f72a7ee282211935a1d82055db2f35ee04c8d692173d02a6cb3a7e6c212243a0R641
動作確認は /save でやった
- https://github.com/kanko-dori/favox/pull/94/files#diff-f72a7ee282211935a1d82055db2f35ee04c8d692173d02a6cb3a7e6c212243a0R16
- Spotify のAPIから拾ってきたJSONを保存できるか検証している
saveSpotifyPlaylist については検証できてないので検証してほしい
- 動作したら、/save を消してマージしたい
Typesをいじったので、フロント部分がすこし変わっている
- Song -> Trackなど
- Spotify の用語に合わせた

ssssota commented 3 years ago

テストデータがいるからだと思うけどデカ目のPRと化している

onsd commented 3 years ago

saveSpotifyPlaylist については検証できてないので検証してほしい動作したら、/save を消してマージしたい

これなので動作したら/save を消すのでサイズは減るはず

ssssota commented 3 years ago

はいぃ…

onsd commented 3 years ago

login できるようになったので動作確認できた動いたが、一部変更しないといけなかったのでissue にする

onsd commented 3 years ago

全体を通してsongとtrackがでてくるの若干気持ち悪いからtrackで統一したほうがいいかもしれんな

わかるのでissue にしよう