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.

Sign up for GitHub

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

Jump to bottom

HOTFIX : Dev to Main #1462

Merged
merged 16 commits into from
Nov 30, 2023
Merged

HOTFIX : Dev to Main #1462

merged 16 commits into from
Nov 30, 2023

Conversation

Ssuamje
Copy link
Collaborator

@Ssuamje Ssuamje commented Nov 30, 2023

해당 사항 (중복 선택)

  • FEAT : 새로운 기능 추가 및 개선
  • FIX : 기존 기능 수정 및 정상 동작을 위한 간단한 추가, 수정사항
  • BUG : 버그 수정
  • REFACTOR : 결과의 변경 없이 코드의 구조를 재조정
  • TEST : 테스트 코드 추가
  • DOCS : 코드가 아닌 문서를 수정한 경우
  • REMOVE : 파일을 삭제하는 작업만 수행
  • RENAME : 파일 또는 폴더명을 수정하거나 위치(경로)를 변경
  • ETC : 이외에 다른 경우 - 어떠한 사항인지 작성해주세요.

설명

아래 링크에 이슈번호를 적어주세요. 예) .../42cabi/issues/738

https://github.com/innovationacademy-kr/42cabi/issues/

연장권 관련한 로직 변경하였습니다.

  • deleted_at으로 소프트 딜리트 이용하던 부분 제거
  • 제거에 따른 로직 수정, 스케줄링 되던 deleteExtensions 메서드 제거.

enaenen and others added 16 commits November 28, 2023 13:42
@enaenen
[BE] FIX: config 수정
c52c646
@Ssuamje
[BE] FIX : LentExtension soft delete 가능하도록 deleted_at 설정
a8be2c5
@enaenen
[BE] TEST: test yaml 수정
994be2c
@Ssuamje
[BE] FIX : deltedAt에 따른 데이터 불러오기 확인
77cc95a
@Ssuamje
[BE] FIX : deleteAt DDL 추가 및 로직 오류 수정
9ea8b02
@Ssuamje
Merge branch 'dev' of github.com:innovationacademy-kr/42cabi into be/…
6e5efd0 
…dev/fix_extension_delete#1457
@Ssuamje
[BE] FIX : 테스트 작성 but 동작 X, 구현 임시 반영
9b9e8d7
@Ssuamje
[BE] FIX : delete 테스트 완료
95affca
@Ssuamje
[BE] FIX : delete 테스트 완료
5b726b5
@Ssuamje
[BE] FIX : delete 테스트 완료
bc01334
@Ssuamje
[BE] FIX : 피드백 반영
5b5aed5
@Ssuamje
Merge pull request #1460 from innovationacademy-kr/be/dev/fix_extensi…
597bada 
…on_delete#1457

[BE] 연장권 삭제 로직 변경 및 변경에 따른 조회 로직 수정
@Ssuamje
[BE] FIX: deleted_at 삭제 및 삭제에 따른 로직, 메서드명 변경
3d4e58b
@Ssuamje
[BE] FIX: deleted_at 삭제, 로직변경 및 테스트
56eaf83
@Ssuamje
[BE] FIX: println 제거
5b2c1fb
@Ssuamje
Merge pull request #1461 from innovationacademy-kr/be/dev/fix_extensi…
fffafc7 
…on_delete#1457

[BE] HOTFIX - deleted_at 제거 및 로직 변경, 반영
@Ssuamje Ssuamje added BE Backend tasks HOTFIX labels Nov 30, 2023
@Ssuamje Ssuamje self-assigned this Nov 30, 2023
@Ssuamje Ssuamje merged commit 342aa75 into main Nov 30, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@enaenen enaenen Awaiting requested review from enaenen

@eunbi9n eunbi9n Awaiting requested review from eunbi9n

@Z1Park Z1Park Awaiting requested review from Z1Park

@sichoi42 sichoi42 Awaiting requested review from sichoi42

@LeeDaeWook LeeDaeWook Awaiting requested review from LeeDaeWook

Assignees

@Ssuamje Ssuamje

Labels
BE Backend tasks HOTFIX
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Ssuamje @enaenen