Skip to content
This repository has been archived by the owner on Aug 24, 2020. It is now read-only.

提出PR #51

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

提出PR #51

wants to merge 25 commits into from

Conversation

ffjlabo
Copy link

@ffjlabo ffjlabo commented Feb 29, 2020

Appeal Point / アピールポイント

期限を過ぎたPRとなってしまい申し訳ありません.
期限を過ぎた上に稚拙なコードになっていてPRを送るか迷ったのですが,せっかくの機会なので提出させていただきました.
現状 css などデザイン部分や UX の向上に関しては着手できていません. プレイリスト一覧表示,動画一覧表示,プレイリストの追加,動画の追加に関して必要最低限の実装となっています.

React + Reduxを使ってstoreでapiから取得したデータを一元管理しました.
また,コンポーネントはできるだけPresentational ComponentとContainer Componentに分けるようにしました.アクセストークンはサーバ側で取得し,Cookieにてフロント側へ持たせるようにしています.

Checklist / チェックリスト
  • Your project fulfills the minimum requirements. / お題の最低要件は満たしている
  • We can build your project. / ビルド方法が記載されている
  • Your poject don't contain (Or we consider it contains) copy-pasted source code. / コピペだと思われるソースコードの仕様をしていない
  • Devices and commitment are described. / 工夫、こだわりが記載されている

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant