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

feature/#49-물품생성 #50

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

HeoJoonSeo
Copy link
Collaborator

#️⃣ 연관된 이슈

feature/#49-물품생성

✅ 체크리스트

  • 🔀 PR 제목의 형식 맞추기 ex) [feat] ㅇㅇ 기능 추가
  • 💡 이슈 등록
  • [ feature] 🏷️ 물품 생성 기능 추가
  • 🧹 불필요한 코드 제거
  • 🧪 로컬 테스트 완료
  • 🏗️ 빌드 성공
  • 💯 테스트 통과

📝 작업 내용

물품 생성 기능을 추가했습니다

여기에 작성

📸 스크린샷 (UI 변경 시 필수)

👀 리뷰어 가이드라인

swagger하고 cmd로 물품 생성되는지 확인해주세요!!

@HeoJoonSeo HeoJoonSeo linked an issue Dec 4, 2024 that may be closed by this pull request
1 task
if (result == 0) {
return "정상적으로 입력되지 않았습니다";
}
return "정상적으로 입력되었습니다";
Copy link
Collaborator

Choose a reason for hiding this comment

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

결과에 대한 문구를 반환해서 문구를 비교하는거 보다,
수업시간에 배운걸 활용해보면 result를 직접 반환해서
result 값으로 비교하는게 더 좋아보여요

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

감사합니다 기능들 다 구현해보고 리팩토링 해보겠습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

물품 생성
2 participants