Skip to content

fix(Budget): 예산 도메인 오류 수정#70

Merged
mipangg merged 1 commit into
devfrom
feat/28-Budget_view
May 21, 2025
Merged

fix(Budget): 예산 도메인 오류 수정#70
mipangg merged 1 commit into
devfrom
feat/28-Budget_view

Conversation

@mipangg

@mipangg mipangg commented May 21, 2025

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈 번호

📝작업 내용

  • 예산 생성 시 평균 환율이 업데이트 되지 않는 점 수정
  • 예산을 중복 생성하지 못하게 수정
  • 예산 수정 시 원화 잔고 값 이상에 대한 수정
  • 예산 수정 후 평균 환율이 업데이트 되지 않는 점 수정
  • 예산 수정을 위해 사용자 입력을 받을 때 환전 여부와 관계 없이 통화 코드는 수정 불가하도록 수정
  • 예산 추가 시 지출과 원화 잔고에 대한 오류 수정

🧪 테스트 여부

  • 테스트 코드를 작성함
  • 테스트를 수행함

💬리뷰 요구사항

  • 리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
  • ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@johnhuh619 johnhuh619 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

확인했습니다.

@pbk2312 pbk2312 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

고생하셨습니다.

@mipangg mipangg merged commit c8d353c into dev May 21, 2025
1 check passed
@mipangg mipangg deleted the feat/28-Budget_view branch May 21, 2025 08:24
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.

3 participants