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] #4

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

Conversation

chs991209
Copy link
Contributor

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

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

    사용자가 수입, 지출 2가지 목록을 확인하고 선택할 수 있게 합니다.

  • 내 PR이 Merge 됨으로써 유저에게 전달되는 편익/기대효과는 무엇일까요?

    사용자는 수입, 지출 2가지 중에서 원하는 유형을 고를 수 있습니다. 선택함으로 인해 서버로 원하는 데이터를 보낼 수 있습니다.


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

  • 수입 지출 여부 목록 조회
    • 금액의 이동의 유형인 "수입" "지출" 2가지 string값을 전부 res.body에 담습니다.
    • 사용자는 브라우저에서는 해당 2가지 목록 중 원하는 것을 골라서 브라우저에서 표시할 수 있습니다.

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


스크린샷 2023-11-14 오전 11 13 20 drop 또는 첨부파일 추가) 스크린샷 2023-11-14 오전 11 29 29

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

  • API 구현만을 위한 최소한의 파일들만 생성하여 올리는 것을 연습하는 데에 도움이 됐습니다.

@chs991209 chs991209 requested a review from soheon-lee November 14, 2023 02:31
@chs991209 chs991209 self-assigned this Nov 14, 2023
@chs991209 chs991209 changed the title Api/수입지출여부목록조회 [50기 최현수 - ADD : Api/수입지출여부목록조회 Nov 14, 2023
Copy link

@soheon-lee soheon-lee left a comment

Choose a reason for hiding this comment

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

마찬가지로 테스트 코드 작성해 주세요

Comment on lines 3 to 11
const getFlowTypes = async () => {
return await appDataSource.query(
`
SELECT id, status as 'option'
FROM flow_type
`
)

}

Choose a reason for hiding this comment

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

Suggested change
const getFlowTypes = async () => {
return await appDataSource.query(
`
SELECT id, status as 'option'
FROM flow_type
`
)
}
const getFlowTypes = async () => {
return await appDataSource.query(
`
SELECT id, status as 'option'
FROM flow_type
`
)
}


const getFlowTypes = async () => {
const flowTypes = await flowTypeDao.getFlowTypes();
if (flowTypes.length===0) {

Choose a reason for hiding this comment

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

Suggested change
if (flowTypes.length===0) {
if (flowTypes.length === 0) {

Comment on lines 9 to 13
else {
return flowTypes;
}
}

Choose a reason for hiding this comment

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

Suggested change
else {
return flowTypes;
}
}
return flowTypes;
}

else 없어도 될 것 같습니다 :)

@chs991209
Copy link
Contributor Author

수정 완료했습니다!

controller {error: error.message || 'INTERNAL ..} 형식으로 수정했습니다.
@chs991209 chs991209 changed the title [50기 최현수 - ADD : Api/수입지출여부목록조회 [50기 최현수 - ADD : Api/수입지출여부목록조회 및 testcode] Nov 21, 2023
@chs991209 chs991209 changed the title [50기 최현수 - ADD : Api/수입지출여부목록조회 및 testcode] [50기 최현수 - ADD : Api/수입지출여부목록조회 리뷰 반영 및 testcode] Nov 21, 2023
@chs991209 chs991209 closed this Jan 4, 2024
@chs991209 chs991209 reopened this Jan 4, 2024
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.

2 participants