🔀 :: (#174) MatchList Page Connect Api

[iloveuhyeon]

💡 개요

match list page에 api 연결했습니다

📃 작업내용

🙋‍♂️ 질문사항

• 개선할 점, 오타, 코드에 이상한 부분이 있다면 Comment 달아주세요.

🍴 사용방법

[Copilot]

Pull Request Overview

This PR connects the match list page to the backend API and integrates state management for loading and displaying match data.

• Adds a new GoRoute for the MatchListScreen with query parameters
• Implements MatchListBloc with event/state to fetch, filter, and sort matches
• Wraps the MatchListScreen in a BlocProvider and updates related UI components

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
lib/router.dart	Registers matchList route with query parameters
lib/presentation/match_team_info/screens/match_team_screen.dart	Wraps existing layout in SafeArea
lib/presentation/match_list/screen/match_list_screen.dart	Integrates MatchListBloc and updates UI for loading and displaying matches
lib/presentation/match_list/bloc/match_list_event.dart	Renames event to LoadItems, adds SortOrder
lib/presentation/match_list/bloc/match_list_bloc.dart	Implements API call, filtering, and sorting logic
lib/presentation/match_detail/widget/match_participant_widget.dart	Fixes incorrect hardcoded redOrBlue parameter
lib/presentation/home/widgets/match_card/match_card_component.dart	Wraps team name Text in Expanded for ellipsis
lib/presentation/home/widgets/appbar/home_appbar.dart	Replaces hardcoded date with DateTime.now()
lib/presentation/community/screen/community_main_screen.dart	Uses LoadingIndicator and refactors filter tap

Comments suppressed due to low confidence (2)

lib/presentation/match_list/bloc/match_list_bloc.dart:27

• Consider adding unit tests for _onLoadItems to verify API integration, filtering, and sorting behaviors.

Future<void> _onLoadItems(

lib/presentation/match_list/bloc/match_list_event.dart:5

• [nitpick] The event name LoadItems is generic. Consider renaming to FetchMatchList or LoadMatchList for clearer intent.

class LoadItems extends MatchListEvent {

[Copilot]

The filter predicate compares event.gameType to itself, which is always true. It should compare each item's gameType, e.g., e.gameType == event.gameType.

Suggested change

	filtered.where((e) => event.gameType == event.gameType).toList();
	filtered.where((e) => e.gameType == event.gameType).toList();