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

企業協賛フィルターモーダルの作成 #825

Merged
merged 24 commits into from
Jul 7, 2024

Conversation

Kubosaka
Copy link
Collaborator

@Kubosaka Kubosaka commented Jul 6, 2024

概要

企業協賛活動のフィルターモーダルを作成した

モーダルのフィルター機能

  • 協賛スタイルのフィルター(複数選択可能)
  • 企業名検索機能(企業名と部分一致)
  • 協賛金回収(有無)
  • ソート

協賛活動ページもリファクタリング行いました

画面スクリーンショット等

 
スクリーンショット 2024-07-06 19 07 56

スクリーンショット 2024-07-06 19 08 23

テスト項目

  • 協賛活動のフィルターが機能しているか]
  • 協賛企業やスタイルを追加してバグを発見する
  • 協賛活動のリファクタリングも行ったので、触ってバグがないか確認お願いします

備考

@Kubosaka Kubosaka self-assigned this Jul 6, 2024
@Kubosaka Kubosaka requested a review from TakuyaNakakura July 6, 2024 10:11
@Kubosaka Kubosaka marked this pull request as ready for review July 6, 2024 10:11
@Kubosaka
Copy link
Collaborator Author

Kubosaka commented Jul 6, 2024

年度の切り替えができなくなってます
直します

@TakuyaNakakura
Copy link
Collaborator

協賛スタイルのチェックボックスが、回収有無の内容を選択すると消えたり残ったりする。
たぶんstate周りの問題な気がします。

@Kubosaka
Copy link
Collaborator Author

Kubosaka commented Jul 6, 2024

@TakuyaNakakura
修正しました

@TakuyaNakakura
Copy link
Collaborator

以下の点について動作確認し、修正を確認しました。

  • 年度による絞り込み
  • 協賛スタイルのチェックボックスのバグ

Copy link
Collaborator

@TakuyaNakakura TakuyaNakakura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Kubosaka Kubosaka requested a review from KazumaSun July 6, 2024 13:43
Copy link
Collaborator

@KazumaSun KazumaSun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多分APIの問題なんだろうけど(まじですみません)、協賛スタイルの条件によって、順番が若干違います。
それ以外の動作は、特に気になりませんでした

  • case 1
    スクリーンショット 2024-07-06 225650
    スクリーンショット 2024-07-06 225741

  • case 2
    スクリーンショット 2024-07-06 225806
    スクリーンショット 2024-07-06 225818

@Kubosaka
Copy link
Collaborator Author

Kubosaka commented Jul 6, 2024

修正しました

Copy link
Collaborator

@KazumaSun KazumaSun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです

@Kubosaka Kubosaka requested a review from TkymHrt July 7, 2024 10:18
Copy link
Collaborator

@TkymHrt TkymHrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Kubosaka Kubosaka merged commit 4c763f3 into develop Jul 7, 2024
@Kubosaka Kubosaka deleted the feat/kubosaka/add_acitivity_filter_modal branch July 7, 2024 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants