-
Notifications
You must be signed in to change notification settings - Fork 155
[2단계 - 상세 정보 & UI/UX 개선하기] 안톨리니 미션 제출합니다. #301
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
Antoliny0919
wants to merge
42
commits into
woowacourse:antoliny0919
Choose a base branch
from
Antoliny0919:step2
base: antoliny0919
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
42 commits
Select commit
Hold shift + click to select a range
a655cf0
refactor: step1 병합 이후 추가 변경사항 반영
Antoliny0919 77b3d21
docs: 기능 요구사항 분석 추가
Antoliny0919 b493754
feat: 영화 클릭시 영화에 대한 자세한 정보가 담긴 모달이 렌더링 되는 기능 추가
Antoliny0919 8a0f978
docs: 필요하지 않은 기능 요구사항 제거
Antoliny0919 2b9f50b
docs: 기능 구현 사항 추가
Antoliny0919 9815964
fix: 영화 클릭시 클릭한 영화에 알맞는 데이터가 렌더링되도록 수정
Antoliny0919 9dbce00
docs: 영화 장르를 가져오는 기능 요구사항 추가
Antoliny0919 f613676
feat: 영화 카테고리를 가져오는 API 추가
Antoliny0919 1142b67
feat: 영화의 자세한정보가 담긴 모달을 닫는 기능 추가
Antoliny0919 c1fe458
fix: 모달이 존재할때 외부가 스크롤 되지 않도록 수정
Antoliny0919 89a3c16
feat: 영화 자세한정보에서 각 섹션별 라벨 추가
Antoliny0919 4713a92
feat: 정적인 모달 추가
Antoliny0919 c8da22d
refactor: 영화 클릭시 다이얼로그가 open되도록 전환
Antoliny0919 ebb21cb
refactor: 다이얼로그를 닫는 이벤트를 1회만 추가하도록 전환
Antoliny0919 3e933a6
refactor: 영화 자세한 정보를 렌더링 하는 방식 전환
Antoliny0919 e7e846b
feat: 내 평점을 기록하는 레이아웃 추가
Antoliny0919 29badd0
feat: 평점을 클릭했을때 로컬스토리지에 개인 평점을 저장하는 기능 추가
Antoliny0919 1e869be
feat: 별점 클릭시 평점 UI를 업데이트하는 기능 추가
Antoliny0919 ecc5a53
docs: 개선 사항 체크
Antoliny0919 b83e537
docs: 리팩토링 사항 추가
Antoliny0919 3cd358a
refactor: 영화 더 가져오는 기능을 무한 스크롤로 전환
Antoliny0919 789897e
fix: 폴더 이름 오타 수정
Antoliny0919 fd0636e
refactor: Main로직을 페이지별로 분리
Antoliny0919 437b3ee
test: 검색 더 보기 테스트 간헐적 실패 수정
Antoliny0919 c769ba0
chore: cypress screenshot ignore 추가
Antoliny0919 9a9511d
refactor: 별점을 문자열로 변환하는 로직 분리
Antoliny0919 621615c
refactor: Renderer로직을 페이지별로 분리
Antoliny0919 8dbfc66
docs: 리팩토링 사항 체크
Antoliny0919 39ce2c6
refactor: 레이팅관련 상수 분리
Antoliny0919 9401033
test: 필요하지 않은 테스트 제거
Antoliny0919 ac08292
fix: 스켈레톤 제거 시 이벤트 리스너 소실되는 현상 수정
Antoliny0919 fb103e3
refactor: 불필요한 getter/setter 패턴 제거
Antoliny0919 1fb9134
refactor: 이미지 경로 모듈 import로 교체, fetch defaultOption 추가
Antoliny0919 f0af170
fix: totalPage를 초기화 하도록 수정
Antoliny0919 eb4b4ac
fix: 필요하지 않은 주석 제거
Antoliny0919 3c5e6d1
fix: 모달 bg가 전체화면 크기가 아닌 현상 수정
Antoliny0919 7122838
fix: setup 네이밍 형식을 일관되게 수정
Antoliny0919 2b9075b
style: 반응형 디자인 추가
Antoliny0919 ae4c2e7
fix: 영화 더 가져오기 이후 이전 영화의 이벤트리스너가 제거된 현상 수정
Antoliny0919 44cdccb
refactor: 중복되는 렌더링 로직 통합
Antoliny0919 1ab4e78
refactor: Renderer 메서드 순서 변경
Antoliny0919 b7f71d8
docs: 기능 요구사항 체크
Antoliny0919 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -24,3 +24,4 @@ dist-ssr | |
| *.sw? | ||
|
|
||
| .env | ||
| cypress/screenshots | ||
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
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.
몇 가지 빠진 시나리오가 있는 것 같아요. "유저가 이 앱에서 할 수 있는 모든 행동"을 기준으로 테스트가 있는지 점검해보면 좋을 것 같습니다!