レビュー(3/21) - Githubissues

miso-devel / use-hearthstone-api

0 stars 0 forks source link

レビュー(3/21) #6

Open wafuwafu13 opened 2 years ago

wafuwafu13 commented 2 years ago

[どっちでもいい]

モックデータは全部同じじゃなくて違うデータを用意することが多いです
card_countは3ですね https://github.com/nisi0929/use-hearthstone-api/blob/0a50f9ee4eb845385243f752c90880e46d223490/src/data/cards.js

[できれば]

構造が同じなので共通化できます
イメージ: https://github.com/Doer-org/official/commit/b45b9cc0826758b44e38a0e53091ad7643d9cef6#diff-25a39a3979fb8b727c20b0b47132ed9f8d974743b690e7e62d3b8dddc2955625L13 https://github.com/nisi0929/use-hearthstone-api/blob/0a50f9ee4eb845385243f752c90880e46d223490/src/components/Cards.jsx#L35-L67

[気になっただけ]

deprecated = 非推奨　なので使うとしてもorder=ascだけで良いのではなかろうか https://github.com/nisi0929/use-hearthstone-api/blob/0a50f9ee4eb845385243f752c90880e46d223490/src/components/Cards.jsx#L18

[細かい]

useは動詞なのでこの場合isがついていたら文法的に違和感 https://github.com/nisi0929/use-hearthstone-api/blob/0a50f9ee4eb845385243f752c90880e46d223490/src/components/Back.jsx#L8

wafuwafu13 commented 2 years ago

[個人的メモ] なくてもいけるっけ...? https://github.com/nisi0929/use-hearthstone-api/blob/0a50f9ee4eb845385243f752c90880e46d223490/src/components/Cards.jsx#L2

miso-devel commented 2 years ago

@wafuwafu13 なくてもいけそうです！！

  const url = new URL(window.location.href).search;
  const query = new URLSearchParams(url);
  que.get("パラメータ名")