y4shiro / uma-card-deck-tools

uma-card-deck-tools.vercel.app
0 stars 0 forks source link

サポカ選択モーダルにて、レアリティやカードタイプで絞り込むと同キャラカードを複数選択出来てしまう不具合を修正 #40

Closed y4shiro closed 1 year ago

y4shiro commented 1 year ago

サポカ選択モーダル (CardModal コンポーネント) にて、レアリティやカードタイプで絞り込むと同キャラの選択禁止フィルタが機能しない状態にある

スクリーンショット 2022-11-28 21 42 12

これは選択禁止フィルタの実装ミスなので修正したい

Todo

y4shiro commented 1 year ago

不具合の原因と修正コード

原因は単純で、サポカに所属するキャラの配列を生成する箇所で全カードのデータではなくフィルター済みのカードデータを利用していた

下記コードの 48 行目にて、本来は全部のサポカデータを利用するはずが、実際にはフィルター済みのサポカデータが利用されている https://github.com/y4shiro/uma-card-deck-tools/blob/ff695ed56a34bbcb3ebb1b0b73eaf591f38b8776/src/features/Support/CardDeck/CardModal/index.tsx#L35-L52

修正したコードは次の通り https://github.com/y4shiro/uma-card-deck-tools/pull/41/files https://github.com/y4shiro/uma-card-deck-tools/blob/fd76261c343a0fa3ed5d4b0e3108a45ea68f1c2b/src/features/Support/CardDeck/CardModal/index.tsx#L35-L52

y4shiro commented 1 year ago

動作確認

修正した結果、カードの絞り込みを行っても同キャラ選択禁止のフィルターが動作するようになった 下記動画を例にすると、レアリティ R のスペシャルウィークを選択すると、他レアリティ / 他タイプのスペシャルウィークのカードが選択できないようになっている

https://user-images.githubusercontent.com/14056951/204792062-78f0c37e-d41d-4417-a2ec-972872a0caa9.mp4

y4shiro commented 1 year ago

merge したので close します