-
Notifications
You must be signed in to change notification settings - Fork 2
Feat(audience): 내 관람 공연 카드 클릭 시 공연 상세 페이지로 이동 구현 #306
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
Open
jin-evergreen
wants to merge
14
commits into
develop
Choose a base branch
from
feat/festival-card-list-routing/#304
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
133fb51
feat: FestivalList 구조 변경 및 핸들러 함수 추가
jin-evergreen 657d482
refactor: 타입 정리
jin-evergreen 68d72a4
feat: 타입 추가
jin-evergreen cb790ef
fix: 빌드 에러 수정
jin-evergreen 9f6614c
refactor: type import로 수정
jin-evergreen 51789eb
refactor: 네이밍 일관성 확보
jin-evergreen f31e637
refactor: 닉네임 관련 로직 통일
jin-evergreen b01e1c4
refactor: 엔드포인트 변경에 따른 네이밍 변경
jin-evergreen d6acd0f
refactor: status 타입 분리
jin-evergreen 57f712d
refactor: generatePath 호출 없이 경로 이동 가능하도록 빌더 도입
jin-evergreen 062ee50
refactor: FestivalList 컴포넌트 삭제
jin-evergreen 26c4e20
fix: dDay로 필드명 통일
jin-evergreen d16e8cd
feat: 경로 별 빌더 추가
jin-evergreen 2089485
refactor: 핸들러 함수 로직을 상위 페이지 컴포넌트로 이동
jin-evergreen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,22 +1,5 @@ | ||
| export interface ViewedFestival { | ||
| festivalId: number; | ||
| title: string; | ||
| mainImageUrl: string; | ||
| period: string; | ||
| status: '관람 중' | '관람 예정' | '관람 완료'; | ||
| wishList: boolean; | ||
| } | ||
|
|
||
| export interface Pagination { | ||
| currentPage: number; | ||
| totalPages: number; | ||
| totalElements: number; | ||
| size: number; | ||
| hasNext: boolean; | ||
| hasPrevious: boolean; | ||
| } | ||
|
|
||
| import type { Festival, PaginationResponse } from './festival'; | ||
| export interface ViewedFestivalsResponse { | ||
| festivals: ViewedFestival[]; | ||
| pagination: Pagination; | ||
| festivals: Festival[]; | ||
| pagination: PaginationResponse; | ||
| } |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion | 🟠 Major
Suggest:
FestivalsResponse는home-response밖으로 분리해 주세요.getAllFestivals와getWishlists는 공용 festival entity API인데, 응답 타입이@shared/types/home-response에 머물면home이라는 페이지 의미가 entity 레이어로 스며듭니다. 이번처럼 재사용 범위가 넓어진 타입은 festival 전용 응답 타입 파일로 옮기거나 API 근처로 이동하는 편이 의존성 방향이 더 자연스럽습니다.As per coding guidelines
apps/**/src/entities/**: Entity 레이어예요. - 단일 도메인 개념을 표현해요. - 페이지/라우트 개념은 금지예요.Also applies to: 15-18
🤖 Prompt for AI Agents