0.0.2 배포 #153
Merged
0.0.2 배포 #153
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
Member
rockpell
commented
Mar 27, 2022
•
rockpell
commented
- ft-auth를 intra-auth로 변경
- 내 좋아요, 내가 작성한 글, 내가 작성한 댓글 페이지네이션 적용
- 카테고리 권한 추가
- 로거 winstom 적용
- DTO 규칙 변경
- 카테고리 권한 베이스 추가
- user 엔티티에 깃헙 관련 아이디 컬럼 이름 수정 - intra id 를 저장하는 컬럼 추가
- 순환참조 오류로 인해서, userrole이 undefined 되는 현상 발견 - 순환 참조를 피하기 위해서 interface로 옮김
- writeableArticle : 게시글 작성/읽기 모두 가능한 권한 설정 - readonlyArticle : 게시글 읽기만 가능한 권한 설정 - writeableComment : 댓글 작성/읽기 모두 가능한 권한 설정 - readonlyComment : 댓글 읽기만 가능한 권한 설정 - reactionable : 리액션을 달수있는 권한 설정 - anonymity : 익명처리가 적용되는지 여부
- ft-auth entity 삭제 - ft-auth를 intra-auth로 바꿈 - 이에 따른 api url 변경
- 게시글 응답에 본인이 쓴글인지 아닌지 - 알 수 있는 isSelf 속성 추가
- 읽기 가능인지를 나타내는 컬럼이므로, readonly 보다는 readable이 적절하다고 판단되어 변경함
- userrole interface가 파일로 따로 나뉘면서, import path 올바르게 수정됨
- base dto 를 만들고, base dto 에서 쓸 예정이라서 삭제
- base dto 생성, swagger 와 validate 적용 - 게시글 생성 dto 생성, base dto에서 PickType 으로 가져옴 - 게시글 조회 dto 는 pagenation 때문에 기존 dto 그대로 사용 - 단, Type 데코레이터로, validate시, 타입 변경 - 게시글 수정 dto 생성, base dto에서 PickType 으로 가져옴 - 변경된 타입 controller, service, repository 에 적용
- skip은 getter 이므로, 역직렬화하지 않도록 class-transformer에서 제공하는 Exclude 데코레이터 사용 - skip은 역직렬화 대상에서 제외
- 역직렬화인경우에만 제외하도록 변경
- 게시글 CRUD 간단한 e2e 테스트 추가 - Dto를 validate 하기위한 validate tester 구현 - utils에 정해진 시간만큼 기다리는 wait 함수 추가
- test 에서 wait 하지 않도록 변경
- user의 intraId를 intraAuth로 이동 - intra 인증 하면 nickname이 인트라 id로 바뀌도록 수정
# Conflicts: # src/intra-auth/intra-auth.controller.ts # test/jest-e2e.json
- user 엔티티에 githubUserName이 추가되어 user를 생성하는 dummy에도 반영하였음
- 알람 res,req 전용으로 새롭게 추가 - 새로운 dto controller, service 에 적용
- 알람 res,req 전용으로 새롭게 추가 - 새로운 dto controller, service 에 적용 - 예외처림함수 never 추가, apinotfoundresponse 추가
- detail-article.dto.ts 제거 - category, user 응답을 전용 response dto를 쓰도록 변경
- article, writer 응답을 전용 response dto를 쓰도록 변경 - my-comment-response 추가
- article, user는 프론트에서 응답값을 사용하지 않기 때문에 삭제
- signin-intra-auth 를 signin-intra-auth-request로 변경
- 인기글 res,req 전용으로 새롭게 추가 - 새로운 dto controller, service 에 적용 - 예외처림함수 never 추가, apinotfoundresponse 추가
- 내가 쓴 댓글에 article entity를 추가 - 기존의 article response dto를 사용하지않고, InnerArticle을 따로 만들어서 사용
- CategoryUserAbilityResponseDto 에서 - CategoryUserAuthResponseDto 로 변경
- github secret을 이용할때 GITHUB으로 시작하는 secret이 존재하면 등록되지 않아서 github 관련 env는 NEST_를 붙여서 등록하였음 - 실제 env로 사용할때는 workflow에서 제대로 들어가도록 지정했기 때문에 문제 없음
Feat: github action을 통한 test workflow 추가
Fix: ft-checkin 서초 필드 오타때문에 값이 0으로 나오는 문제 수정
Hotfix: pagination skip 오류 수정
- 페이지네이션에서 skip 속성을 뺐는데, test 코드에는 남아있어서 오류가 발생함 - 이제 제거 했어요 ㅎㅎ
- push 시에는 이벤트 발동하지 않도록 수정 - 브랜치 develop 뿐만 아니라 전체 발동 되도록 수정
- find-one-article-response 생성 - 대신에 기존에 article-response에는 isSelf, isLike 제거 - 다른 응답에서는 필요없는 속성이므로 제거하였습니다.
Hotfix : pagination 테스트 에러 수정 및 깃허브 액션 수정
- api 응답별로 response dto를 분할하였습니다. - 게시글 목록 반환에 카테고리 응답 제거
- 기존의 CategoryResponseDto를 삭제하고, - 카테고리 권한 연산 결과를 응답하도록 변경
Feat: �User, Category, Comment, Notification, Reaction dto 설정
Skyrich2000
approved these changes
Mar 27, 2022
mimseong
approved these changes
Mar 27, 2022
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.