Skip to content

[#264] feat: 우선순위 변경 로직 수정 및 테스트코드#265

Closed
mmihye wants to merge 1 commit intodevelopfrom
feature/#264
Closed

[#264] feat: 우선순위 변경 로직 수정 및 테스트코드#265
mmihye wants to merge 1 commit intodevelopfrom
feature/#264

Conversation

@mmihye
Copy link
Member

@mmihye mmihye commented Apr 4, 2025

🚩 관련 이슈

📋 구현 기능 명세

  • 카테고리 수정시 락 설정

📌 PR Point

  • 무슨 이유로 어떻게 코드를 변경했는지
    카테고리 수정 시 데드락 발생

  • 어떤 부분에 리뷰어가 집중해야 하는지
    auto format 때문에 변경사항 보기 불편할거같아서 중요 지점만 코멘트 남겨둡니다.

  • 개발하면서 어떤 점이 궁금했는지

📸 결과물 스크린샷

테스트 코드 (로그 있는 버전 - 지금은 로그 삭제)
동시에 실행시 앞에 트랜잭션이 완료될때까지 기다림
스크린샷 2025-04-04 오전 11 52 49

🛠️ 테스트

  • 테스트

🚀 API Endpoint

@mmihye mmihye requested a review from sss4920 April 4, 2025 03:16
@mmihye mmihye self-assigned this Apr 4, 2025
import jakarta.validation.constraints.NotNull;

public record ChangeCateoryPriorityDto(
public record ChangeCategoryPriorityDto(
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오타수정

@mmihye mmihye closed this Apr 4, 2025
@mmihye mmihye deleted the feature/#264 branch April 4, 2025 03:20
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.

[Feature] 카테고리 변경로직 락

1 participant