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

[Refactor/66] 항목 순서변경 API 수정 #67

Merged
merged 1 commit into from
Jul 29, 2024
Merged

[Refactor/66] 항목 순서변경 API 수정 #67

merged 1 commit into from
Jul 29, 2024

Conversation

BaekJaehyuk
Copy link
Contributor

@BaekJaehyuk BaekJaehyuk commented Jul 29, 2024 •
edited
Loading

🚀 개요

  1. 보관함에 존재하는 비활성화 항목도 순서변경 가능하도록 수정
  2. 순서 변경 시, 변경할 순서에 위치한 항목과 서로 변경했던 로직 수정

🔍 변경사항

  • 보관함에 존재하는 비활성화 항목도 순서변경 가능하도록 수정
  • 순서 변경 시, 순서를 서로 변경했던 로직에서 항목들을 순차적으로 재배치하도록 수정
  • 에러 코드 추가

⏳ 작업 내용

  • Service 로직 수정
  • ErrorStatus 추가

📝 논의사항

@BaekJaehyuk BaekJaehyuk self-assigned this Jul 29, 2024
@BaekJaehyuk BaekJaehyuk linked an issue Jul 29, 2024 that may be closed by this pull request
♻️ [Refactor] 항목 순서변경 API 수정 #66
Closed
2 tasks
@BaekJaehyuk BaekJaehyuk merged commit c76debe into develop Jul 29, 2024
@BaekJaehyuk BaekJaehyuk deleted the Refactor/66 branch July 29, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Eunjin3395 Eunjin3395 Eunjin3395 approved these changes

Assignees

@BaekJaehyuk BaekJaehyuk

Labels
♻️ refactor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

♻️ [Refactor] 항목 순서변경 API 수정
2 participants
@BaekJaehyuk @Eunjin3395