[1단계 - 나만의 유튜브 강의실] 심바(임선빈) 미션 제출합니다. #6
Merged
Conversation
This file contains 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
|
페어의 리뷰도 반영하여 함께 올렸습니다. |
ysm-dev
pushed a commit
that referenced
this pull request
Mar 16, 2021
* [1단계 - 나만의 유튜브 강의실] 심바(임선빈) 미션 제출합니다. (#6) * chore: eslint, prettier, cypress 세팅 * docs: todo 작성 * chore: cypress 세팅 * test: 클릭한 탭의 색 변경 테스트 작성 * feat: dom 조작 라이브러리, View 기본 클래스 작성 * feat: dom 조작 라이브러리, View 기본 클래스 작성 * refactor: dom 라이브러리, View 클래스 수정 * feat: 네비게이션 탭 이동 시 버튼 색상 변경 기능 구현 * test: 동영상 검색 버튼 클릭 시 검색 모달 창 열리는 테스트 작성 * feat: 동영상 검색 버튼을 누르면 검색 모달 창이 열림 * test: 검색 모달 창의 x 버튼 클릭 시 모달이 닫히고, 볼 영상 목록으로 돌아가는 테스트 작성 * feat: 검색 모달 창의 x 버튼을 누르면 모달 창이 닫히고, 볼 영상 목록으로 돌아가는 기능 작성 * chore: api key 저장 * test: 현재 검색한 검색어가 최근 검색어 목록에 남는 테스트 작성 * feat: 검색 시 최근 검색어 3개가 남는 기능 구현 * feat: 검색 시 api 결과를 받아서 Video 인스턴스를 생성하는 기능 구현 * feat: 검색 실행 시 검색 결과를 10개씩 가져와서 보여주는 기능 구현 * test: 검색 결과를 가져오는 동안 skeleton UI를 보여주는 테스트 작성 * feat: 검색 결과를 가져오는 동안 skeleton UI를 보여주는 기능 구현 * test: 검색 결과가 없는 경우 결과 없음 이미지를 보여주는 테스트 작성 * feat: 검색 결과가 없는 경우 결과 없음 이미지를 보여주는 기능 구현 * test: 검색 후 스크롤을 내리면 검색 결과를 10개씩 더 보여주는 테스트 작성 * feat: 검색 후 스크롤을 내릴 경우 검색 결과를 10개씩 더 가져오는 기능 구현 * feat: 모달 창을 열면 가장 마지막에 검색한 검색 결과 동영상들을 보여주는 기능 구현 * refactor: Video 모델 필드 프라이빗, 날짜 출력 형식 변경 * style: 모달 검색 클립 저장 버튼 추가 * feat: 검색 결과 동영상 저장 기능 구현 * refactor: 네비게이션 탭 custom dom library event로 변경 * feat: 저장한 동영상들을 볼 영상 목록에 추가하는 기능 구현 * test: 검색 결과 동영상을 저장하면 동영상을 볼 영상 목록에 추가하는 테스트 작성 * feat: 동영상 저장 시 해당 동영상의 저장 버튼 비활성화, 저장 개수 업데이트 기능 구현 * fix: 모달 스크롤 시 최근 검색어를 유지하고, 모달을 닫을 때 검색 내역 삭제하도록 수정 * fix: 검색어를 입력하지 않을 시 경고창 띄우도록 수정 * feat: 최근 검색어 클릭 시 해당 검색어로 검색하는 기능 구현 * fix: 모달 스크롤 시에만 nextPageToken 값을 유지하도록 수정 * test: 최근 검색어를 누르면 해당 검색어로 검색을 실행한 결과를 보여주는 테스트 작성 * fix: 검색 후 모달 결과 창의 스크롤를 최상단으로 이동 * refator: 테스트 코드에서 중복되는 부분 함수로 분리 * style: - EOL 수정 - 함수 컨벤션 통일 - 주석 제거 * style: buttom type에서 submit 제거 * refactor: Nav Tab에서 버튼 별로 기능 분리 * refactor: api URL 생성 함수 분리 * stlye: 태그 속성 추가 * refactor: - localStorage key 값 상수화 - json string 검증 * chore: vscode 세팅 gitignore 추가 * chore: vscode 세팅 gitignore 추가+ * chore: vscode 세팅 gitignore 추가 * refactor: URL 제작 방식 URLSearchParams로 수정 * refactor: - dom library 불필요한 리턴 제거 - scroll throttle 함수 분리 * refactor: callback을 async-await로 변경 * refactor: - img 태그에 alt 추가 - dom library Error 상황 추가 - cypress api 요청 기다리는 함수, 실제 키워드 검색 테스트 추가 * refactor: localStorage array 저장 방식 변경 Co-authored-by: zigsong <wldms5764@gmail.com> * docs: 2단계 Todo 작성 * test: 저장된 동영상이 없으면 비어있음을 알리는 이미지를 보여주는 테스트 작성 * feat: 저장된 영상이 없으면 비어있음을 알려주는 이미지를 보여주는 기능 구현 * test: 영상을 100개 이상 저장 시 alert을 띄우는 테스트 작성 * feat: 영상을 100개 이상 저장 시 alert을 띄우는 기능 구현 * refactor: Store와 observer pattern 적용 * test: 동영상 저장 시 성공/실패 여부를 알려주는 snackbar를 띄우는 테스트 작성 * feat: 동영상 저장 시 성공/실패 여부를 알려주는 snackbar를 띄우는 기능 구현 * refactor: localStorage 저장 방식 변경 * test: 본 영상을 체크하면 버튼이 표시되고 snackbar를 띄우는 테스트 작성 * feat: 본 영상을 체크하면 버튼이 표시되고 snackbar를 띄우는 기능 구현 * test: 동영상 삭제 버튼 클릭 시 confirm을 띄우고, 동의 시 삭제하는 테스트 작성 * feat: 동영상 삭제 버튼 클릭 시 confirm을 띄우고, 동의 시 삭제하는 기능 구현 * test: 각 영상 탭에 맞는 영상 목록을 보여주는 테스트 작성 * feat: 각 영상 탭에 맞는 영상 목록을 보여주는 기능 구현 * style: snackbar css 변경 * style: clip transition css 추가 * refactor: 볼 영상 탭에서 본 영상 제거 * test: snackbar와 url 주소 변경 * fix: 본 영상 목록에서 삭제시 데이터가 남아있던 문제 해결 * refactor: 동동's API 서버 적용 * feat: 각 탭에서 동영상 목록이 비게 되면 텅 이미지를 보여주는 기능 구현 * refactor: validator 분리 * refactor: - 비즈니스 로직 순서 변경 - App 로드 방식 변경 - keyword 데이터 뷰에서 컨트롤러로 이동 - 이벤트 위임 적용 * refactor: localStorage 기본값 리턴 로직 수정 * fix: 탭 간 이동 또는 ✅ 버튼 체크 시 재생 중인 동영상 정지 * refactor: - separate url constant in cypress - remove array destructuring in controller - handle error in http request * refactor: - handle empty search keyword & trimming - handle videoToSave undefined case - name variable in video button pack dataset objects * refactor: update localStorage methods * refactor: - update css transition - rename scrollEnd variable * refactor: update checking current tab logic Co-authored-by: 0imbean0 <75007375+0imbean0@users.noreply.github.com> Co-authored-by: 0imbean0 <tjsqls2003@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
안녕하세요.
우아한테크코스 3기 심바입니다.
1단계 미션 제출합니다.
이번 미션에서는 이전 미션들보다 Model, View, Controller의 기능을 명확하게 분리하기 위해 고민했습니다.
데모 사이트 - 나만의 유튜브 강의실
📝 stpe1 기능 구현
검색 결과 스크롤
동영상 저장
최근 검색어 클릭
검색 결과 없음