비공유 상태일 시 code null값 반영 및 sharedTrip soft delete 추가 #530
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
📝 Jacoco Test Coverage
|
LJW25
requested changes
Sep 5, 2023
| @@ -36,8 +34,7 @@ public static TripDetailResponse of(final Trip trip, final List<City> cities) { | |||
| .map(CityWithPositionResponse::of) | |||
| .toList(); | |||
|
|
|||
| final Optional<SharedTrip> sharedTrip = Optional.ofNullable(trip.getSharedTrip()); | |||
| final String sharedCode = sharedTrip.map(SharedTrip::getSharedCode).orElse(null); | |||
| String sharedCode = trip.getSharedCode().orElse(null); | |||
hgo641
reviewed
Sep 5, 2023
Comment on lines
158
to
159
| .header(AUTHORIZATION, | ||
| "Bearer " + memberTokens.getAccessToken()) |
Comment on lines
169
to
170
| .header(AUTHORIZATION, | ||
| "Bearer " + memberTokens.getAccessToken()) |
LJW25
approved these changes
Sep 6, 2023
hgo641
approved these changes
Sep 6, 2023
waterricecake
force-pushed
the
feature/#520-shareTripDelete
branch
from
0f3af9f
to
49f59b8
Compare
waterricecake
force-pushed
the
feature/#520-shareTripDelete
branch
from
e6a5d6d
to
5e3fdcd
Compare
jjongwa
pushed a commit
that referenced
this pull request
Sep 12, 2023
* feat: soft delete 추가 * refactor: 메서드 반환 타입 변경 * style : 코드 컨밴션 적용
jjongwa
added a commit
that referenced
this pull request
Sep 20, 2023
* city 패키지 위치 변경 (#527) * ExpensePage 리팩토링 (#533) * refactor: 상수화 * refactor: ExpensePage 관련 코드 리팩토링 * refactor: 오타 수정 * refactor: createTrip 테스트 코드 수정 * TripPage, TripEditPage 리팩토링 (#537) * fix: TripEditPage 데이터 패칭 waterfall 문제 해결 * refactor: useQuery 옵션 defaultOptions로 옮기기 * fix: Header cursor pointer 문제 해결 * refactor: Footer 수정 * refactor: 회원가입, 로그인 버튼 하나로 합치기 * refactor: props로 넘겨주는 대신 TripInformation에서 직접 데이터 가져오기 * refactor: TripInformation 수정 버튼 네이밍 수정에서 완료로 변경 * refactor: TripItem 제목 스타일링 수정 * refactor: 모바일일때 image carousel navigation 보이게 변경 * refactor: 수정, 삭제 버튼 더보기 메뉴에서 보여주는 대신 아이콘으로 변경 * refactor: 기타 탭에서 아이템 생성 시 디폴트 카테고리 기타로 설정 * refactor: 이미지 용량 10mb 이상으로 인한 에러 발생 시 상황에 맞는 에러 메세지 보여주기 * refactor: 아이템 추가할 때 제목에 공백 입력 후 아이템 추가 시 에러 메세지 보여주기 * refactor: TripItemList 컴포넌트 수정 * fix: 테스트 코드 수정 * 삭제 컨펌 경고 모달 추가 (#540) * refactor: 여행 삭제 시 삭제 컨펌 모달 보여주기 * refactor: 여행 아이템 삭제 시 삭제 컨펌 모달 보여주기 * MyPage 리팩토링 (#541) * refactor: 탈퇴 버튼 스타일링 수정 * refactor: 계정 탈퇴 클릭 시 컨펌 모달 보여주기 * 금액순으로 카테고리 정렬 구현 (#531) * feat: 금액순으로 카테고리 정렬 구현 * style : 코드 컨밴션 적용 * 번들 사이즈 최적화 (#546) * chore: webpack bundle analyzer 설치 * chore: compress webpack plugin 설치 * refactor: 이미지 파일 최적화 * refactor: tree shaking 추가 * refactor: tsconfig.json module commonjs에서 esnext로 변경 * refactor: lazy loading을 활용한 코드 스플리팅 * refactor: IntroPage 이미지 png에서 jpg으로 변경 * refactor: 탈퇴 버튼 수정 * fix: npm run build 시 오래 걸리는 문제 해결 (#549) * 모바일 환경에서 StarRatingInput 호버 문제 수정 (#550) * refactor: 디자인시스템 1.3.0 버전업 * refactor: 모바일 환경에서 StarRatingInput 호버 문제 수정 * FLYWAY 적용 (#552) * chore: flyway 적용 * chore: submodule 변경 적용 * chore: submodule 변경 적용 * 모바일 버전에서 지도 볼 수 있는 기능 (#544) * feat: 모바일에서 지도보기 버튼 추가, 지도 보는 기능 * refactor: 트립페이지 모바일 버전 컴포넌트 분리 * feat: 모바일에서 지도볼때도 daylog tab보이도록 수정 * refactor: 버튼 스타일 수정, 패딩 수정 * chore: 리베이스로 인한 버그 수정 * 비공유 상태일 시 code null값 반영 및 sharedTrip soft delete 추가 (#530) * feat: soft delete 추가 * refactor: 메서드 반환 타입 변경 * style : 코드 컨밴션 적용 * 토글버튼 부활, UI관련 수정 (#557) * feat:데이로그 아이템 토글 기능 부활 * feat:이미지 클릭시 확대 모달창 여는 기능 * refactor: 지도 로딩시 스피너 중앙에 위치 * refactor: 이미지 사이즈 조정 로직 훅으로 분리 * refactor: 제목길이 25로 늘림 * refactor: api통신 횟수 조정, queryClient 함수 분리 * refactor: 여행설명, 아이템메모 사용자입력 줄바꿈 적용 * sharedtrip 변경사항 flyway script 작성 (#556) * feat: sharedtrip 변경사항 flyway script 작성 * feat: sharedtrip 변경사항 flyway script 작성 * refactor: 스크립트 명 변경 * TripCity 삭제 로직 추가 (#529) * refactor: 테스트코드 패키지 변경 (#563) * webpack 최적화 및 common, prod, dev 파일 분리 (#560) * refactor: webpack 최적화, dev/prod파일 분리 * refactor: webpack serve, build script수정 * refactor: webpack파일 분리로 인한 cypress 자동화 yml 수정 * 접속자별 접근 권한 검증 구현 (#571) * feat: AuthArgumentResolver 로직 추가 Co-authored-by: mcodnjs <[email protected]> * feat: MemberOnly 어노테이션 구현 Co-authored-by: mcodnjs <[email protected]> * feat: MemberOnly 어노테이션 적용 Co-authored-by: mcodnjs <[email protected]> * fix: cookie null 예외처리 추가 Co-authored-by: mcodnjs <[email protected]> --------- Co-authored-by: mcodnjs <[email protected]> * 닉네임 랜덤 숫자 추가 및 수정 시 닉네임 중복 체크 (#565) * feat: member의 nickname에 unique 제약조건 추가 * feat: 회원가입 시 닉네임 랜덤넘버 추가 * feat: 닉네임 수정 시 중복 체크 로직 추가 * refactor: 랜덤 생성 로직 수정 및 상수화 * Trip, Like, PublishedTrip 테이블 생성 (#569) * feat: like 엔티티 구현 * feat: publishedTrip 엔티티 구현 * feat: flyway 마이그레이션 스크립트 작성 * feat: 기본 생성시 unpublished 추가 * style: 코드 컨벤션 적용 * style: 코드 컨벤션 적용 * refactor: flyway 스크립트 버전 수정 * 이미지 압축 (#570) * chore: browser image compression 설치 * feat: 이미지 최적화 추가 * refactor: preview 이미지 보여주는 방법 수정 * 마이페이지 수정 시 닉네임 변경되지 않은 경우 예외 발생 버그 해결 (#576) * fix: 닉네임 변경 시에만 중복 닉네임 검사 로직 추가 * refactor: 멤버 수정 날짜 업데이트 로직 추가 * 여행 아이템 사진 확대해서 볼 수 있는 기능 추가 (#580) * chore: 행록 디자인 시스템 버전 업데이트 * feat: 이미지 확대 모달 구현 * 성능 최적화 - 사용자 이미지 아이콘 width, height 추가 (#586) * fix: 사용자 이미지 아이콘 width, height 추가 * chore: compression 관련 코드 삭제 * 경비 공유 페이지 생성 (#559) * feat: 공유된 페이지에 가계부 버튼 노출 * fix: 메모, 상세설명 줄바꿈 되도록 수정 * feat: 가계부 공부 페이지 api 구현 * feat: 공유 가계부에서 공유 트립으로 이동 * refactor:버그 수정, 여행추가 버튼 위치 변경 * refactor:지도 확대 가능, 지도 클릭 불가능 기능 * feat:공유페이지 모바일 버전 적용 * refactor: 파일 명 변경 * refactor: 로그인시 메인이 아닌 전 페이지로 이동 * fix: 비로그인 시 공유경비 페이지 못보는 오류 해결 * refactor:디자인 변경, 변수명 리네임 * refactor: 여행정보사진수정 커스텀 훅 multi->single로 변경 * refactor:버그로 인한 이미지 업로드 롤백 * fix:CF배포로 인한 api 주소 변경 (#588) * fix:dev서버에서 백서버로 api요청 안가는 오류 해결 ->axios base url환경변수 변경 (#594) * 공유된 여행 가계부 조회 기능 추가 (#595) * refactor: sharedCode index 설정 추가 (#591) Co-authored-by: hgo641 <[email protected]> Co-authored-by: jjongwa <[email protected]> * feat: cache-control 메타데이터 추가 (#597) * refactor: cache-control 메타데이터 값 변경 (#600) * cors origin과 method 추가 (#603) * RefreshToken memberId Unique 해제 (#606) --------- Co-authored-by: Ashley Heo <[email protected]> Co-authored-by: waterricecake <[email protected]> Co-authored-by: 임우찬 <[email protected]> Co-authored-by: 이지우 <[email protected]> Co-authored-by: Dahye Yun <[email protected]> Co-authored-by: mcodnjs <[email protected]> Co-authored-by: Chaewon Moon <[email protected]> Co-authored-by: hgo641 <[email protected]>
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.
📄 Summary
또한 비공유 상태일 시 null 값을 넣어 front 측에 보내주었습니다.
🙋🏻 More