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

v1.1.1 #429

Merged
merged 13 commits into from
Oct 17, 2022
Merged

v1.1.1 #429

merged 13 commits into from
Oct 17, 2022

Conversation

nan-noo
Copy link
Collaborator

@nan-noo nan-noo commented Oct 17, 2022

요약

  • 버그 수정

nan-noo and others added 13 commits October 10, 2022 22:15
* refactor: is<Type>함수 typeChecker로 통합

* refactor: handlers 폴더로 묶기

* feat: typeChecker 구현

* feat: json 파일 수정

* feat: handler 수정

* feat: tag type check

* feat: member type check

* feat: link type check

* feat: link-preview type check

* feat: auth type check

* feat: community & notice article type check

* feat: review type check

* feat: study, my study type check

* feat: Owner type 수정

* feat: axios response interceptor 수정

아무 응답도 오지 않을 시 빈 string('')이 응답 데이터로 반환됨
-> 명시적으로 null로 교체

* refactor: TODO 주석 추가

* refactor: checker 타입 수정 및 함수 위치 이동

- 도메인에 맞게 이동

* refactor: 타입 이름 변경 및 import 'type' 추가

* refactor: axiosInstance interceptor 함수 분리

* feat: util typeChecker axios 의존성 제거

* refactor: my study 함수명 및 타입명 수정

* feat: checkOptionalType과 checkType 합치기

* feat: link preview 타입 수정

* refactor: util typeChecker 수정

* refactor: 상수 분리

* feat: key 배열이 객체의 모든 key를 담고 있는지 확인

* feat: object.hasOwnProperties -> Object.hasOwn

- https://eslint.org/docs/latest/rules/no-prototype-builtins
- https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/hasOwn

* fix: 라우팅 오류 수정

* refactor: 상수 분리

- USER_ROLE
- CATEGORY_NAME
- STUDY_STATUS

* feat: checkType 함수 수정

isNullOrUndefined -> isNull
- main page 스터디 목록
- my study page 스터디 목록
-> 타입 충돌 오류 수정
* fix: 스터디 상세 페이지 소개글 overflow시 가로 스크롤되도록 수정

* fix: 글쓰기/미리보기 컴포넌트 overflow시 스크롤 되도록 수정

* fix: subject tag validation 조건 추가

- 한 개 이상을 선택해야 생성할 수 있도록 변경

* feat: 누구나 스터디방 페이지에 접근할 수 있도록 수정

- 상세 페이지 가입하지 않았거나 모집완료일 때 이동하기 버튼 보여주기
- 스터디 방 페이지 접속시 Read만 가능하도록 수정

* fix: maxMemberCount가 string 타입으로 전송되는 오류 수정

* feat: 로그인 성공시 이전 페이지로 이동하도록 수정

* fix: 스터디 종료 날짜보다 스터디 모집 마감 날짜가 이후인 오류 수정

* feat: 스터디 수정/생성 submit 시 오류가 있으면 alert하도록 수정

* test: history pop시 cypress CORS 에러 수정

- 테스트 오류 원인: 로그인 페이지 접속 후 로그인에 성공하면 뒤 페이지로 이동하는데, cypress가 이 때문에 CORS에러를 발생시킴
- 테스트 오류 해결: 로그인 페이지 접속하지 않고 액세스 토큰을 직접 설정

* test: cypress 오류 수정
* feat: user role, user info api 수정

* feat: 로그인 관련 훅 수정

- useAuth
- useUserInfo
- useUserRole

enabled 옵션 변경

* feat: 로그인 관련 훅 사용하고 있는 컴포넌트 수정

* refactor: type과 상대경로 수정
* fix: 로그인시 헤더가 바뀌지 않는 문제 해결

* refactor: eslint 적용
@2022-moamoa
Copy link

2022-moamoa bot commented Oct 17, 2022

Passed

Analysis Details

0 Issues

  • Bug 0 Bugs
  • Vulnerability 0 Vulnerabilities
  • Code Smell 0 Code Smells

Coverage and Duplications

  • Coverage No coverage information (70.30% Estimated after merge)
  • Duplications No duplication information (0.40% Estimated after merge)

Project ID: woowacourse-teams_2022-moamoa_AYKvd_z4VbW_bWBvgn13

View in SonarQube

@nan-noo nan-noo merged commit b4d45a4 into main Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants