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

[50기 최현수 - ADD : Api/용돈조회등록수정삭제 및 잔여 용돈 조회 및 testcode] #7

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

chs991209
Copy link
Contributor

@chs991209 chs991209 commented Nov 16, 2023

1. 본 PR이 우리 팀의 웹 서비스 제품성에 어떠한 기여를 하였고,
사용자에게 어떠한 기대효과를 전달하는지 작성해주세요.

  • 내 PR이 제품 내 어떠한 기능적인 배경/전후맥락 가운데 개발되었나요?

    • 가족의 관리자가 본인 또는 가족들에게 각 달마다 용돈을 할당하고 가족에게 보여줄 수 있습니다.
      • (단, 같은 유저가 같은 달의 중복 용돈이 할당되는 것은 백엔드 서버에서 차단하여 불가능합니다.)
    • 또한 수정, 삭제가 가능합니다.
    • 가족 구성원 중 특정 유저의 특정 연월의 용돈에서 개인의 지출 금액 합산을 차감한, 잔여 용돈량을 조회할 수 있습니다.
  • 내 PR이 Merge 됨으로써 유저에게 전달되는 편익/기대효과는 무엇일까요?

    • 사용자는 원하는 시기에 용돈을 달라고 관리자(대표자)에게 부탁하면, 관리자를 통해 연, 월과 금액을 지정하여 용돈을 할당받으며,
    • 다른 가족의 용돈까지 확인할 수 있습니다.
    • 또한, 용돈 수정 api는 금액만 수정하게 하는 방식으로 백엔드 서버 api를 구현하여 중복된 연, 월의 용돈 에러를 사전에 방지하였습니다.
    • 가족 구성원에게 할당된 용돈에서 구성원 별 용돈 대비 지출 금액을 알 수 있습니다.

2. 이 브랜치에서 어떤 내용을 개발했는지 큰 제목과 상세 내역을 적어주세요.

용돈 조회 등록 수정 삭제 및 잔여 용돈 조회

  • 용돈 등록 => 특정 가족원에게 줄 용돈의 연, 월, 금액을 등록하여, 해당 가족 유저기존에 등록했던 용돈과 중복되지 않는 연, 월의 용돈을
    • 등록할 수 있습니다. 한 번에 하나만 등록할 수 있습니다. (관리자 권한 필수)
  • 용돈 수정 => 금액만 수정할 수 있습니다. (관리자 권한 필수)
  • 용돈 조회 => 접속한 유저가 속한 모든 가족의 용돈을 조회할 수 있습니다. Query로 특정 가족 구성원의 이름, 연, 월을 요청하여
    • 특정 가족 구성원의 특정 시기의 용돈을 조회할 수 있습니다
    • (단, 월 조건만 요청하면 안됩니다. ex. 7월의 용돈 보여줘 => error, 23년 7월의 용돈 보여줘 => GET_SUCCESS)
  • 용돈 삭제 => 기존에 존재하는 용돈을 삭제할 수 있습니다. 이미 삭제됐거나 존재하지 않는 용돈이면 에러를 반환합니다. (관리자 권한 필수)
  • 잔여 용돈 조회 => 특정 유저의 특정 연,월의 용돈 - 해당 유저의 해당 연, 월 지출 합산 금액 (연, 월 조건 query 필수)

변동사항

  • req.body의 amount: 10000 등을 allowance: 10000으로 변경했습니다. 용돈 그 자체라는 의미로 바꿨습니다.

기타 수정 => browser에서 Request에 담은 id를 지표로 용돈을 수정, 삭제하는 함수 추가 및 주석 처리하였습니다.
프론트엔드에서 구현 방식에 대해 정해지면 두 방식 중 하나를 활용할 계획입니다(브라우저에서 용돈의 목록에서 id를 들고 있다가 req.body에 담아서 보낼 건지, req.body에 보내지 않을 건지 미정입니다).
=> 일부 수정 => 현재 query로 id 값을 받아 해당 id를 지표로 삭제하는 것으로 진행될 예정입니다.


3. 개발한 화면을 캡쳐해서 첨부 해 주세요.


![스크린샷 2023-11-22 오전 4 39 39](https://github.com/wecode-bootcamp-korea/50-3rd-Fin_Pong-backend/assets/82556127/d371623c-f4aa-42dd-bde4-bec1eb129ce6)

4. 이 브랜치에서 개발하면서 느꼇던 개발 성장포인트를 적어주세요.

  • 에러 헨들링으로 사용자에게 편의를 주는 설정 방법에 배웠으며, DB protocol과 SQL의 data의 빌드 방식에 대해서 더 공부하고 싶어졌습니다.
  • 차트의 데이터 별 id를 브라우저에서 쥐고 있다가 선택해서 수정하고자 할 때 http request로 보내는 것의 난이도?에 대해 궁금해졌습니다.

@chs991209 chs991209 requested a review from soheon-lee November 16, 2023 17:30
@chs991209 chs991209 self-assigned this Nov 16, 2023
@chs991209 chs991209 changed the title [50기 최현수 - ADD : 용돈조회등록수정삭제] [50기 최현수 - ADD : Api/용돈조회등록수정삭제] Nov 16, 2023
@chs991209 chs991209 changed the title [50기 최현수 - ADD : Api/용돈조회등록수정삭제] [50기 최현수 - ADD : Api/용돈조회등록수정삭제 및 잔여 용돈 조회] Nov 19, 2023
@chs991209 chs991209 changed the title [50기 최현수 - ADD : Api/용돈조회등록수정삭제 및 잔여 용돈 조회] [50기 최현수 - ADD : Api/용돈조회등록수정삭제 및 잔여 용돈 조회 및 testcode] Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant