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.

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

[BE] 카테고리 버그 수정 #1123

Open
wants to merge 1 commit into
base: BE/dev
Choose a base branch
from
Open

[BE] 카테고리 버그 수정 #1123

wants to merge 1 commit into from

Conversation

yechop
Copy link
Contributor

@yechop yechop commented Feb 12, 2025

연관된 이슈

구현한 기능

상세 설명

이전 PR(#1116)에서 카테고리 이름을 수정할때,
수정할 카테고리 이름의 길이를 검증하고자 new Category(수정할 카테고리 이름)를 추가해야 하는데
실수로 new Category(기존 카테고리 이름) 으로 코드를 작성했습니다.

new Category(수정할 카테고리 이름) 안에서 수정할 카테고리 이름에 대한 길이 검증을 하고 있습니다.

@yechop yechop self-assigned this Feb 12, 2025
@@ -59,7 +58,6 @@ void show_category() {
.statusCode(200);
}

@Disabled
Copy link
Member

Choose a reason for hiding this comment

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

테스트 코드도 다시 손좀 봐줘야할텐데 다음 오프때는 테스트코드도 건들여보죠

Comment on lines +67 to 69
final Category category = new Category(request.value());
categoryEntity.updateCategoryName(category);
return new CategoryUpdateResponse(categoryEntity.getCategoryName());
Copy link
Member

Choose a reason for hiding this comment

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

아래 리턴에는 request.value()로 안바꿔줘도 되는건가요? 리턴값은 사용하지 않아서 상관 없으려나요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

이번 PR 내용이 별로 없어 이해가 안되실것 같아 PR 상세 설명에 �버그 픽스 내용 적어놓을게요
일단 리턴값은 지금 상태가 맞습니다

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.

2 participants