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

안부전화 신청하기 포인트 계산 #87

Merged
merged 27 commits into from
Oct 20, 2024
Merged

안부전화 신청하기 포인트 계산 #87

merged 27 commits into from
Oct 20, 2024

Conversation

Dobbymin
Copy link
Contributor

#️⃣ 연관된 이슈

ex) #이슈번호, - #이슈번호

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요.(이미지 첨부 가능)

  • 안부전화 포인트 계산 기능 구현

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

⏰ 현재 버그

✏ Git Close

close #이슈번호

@Dobbymin Dobbymin added ✨ Feature 새로운 기능 추가 및 구현하는 경우 📡 API 비동기 통신 코드를 짜는 경우, 백엔드와의 통신하는 경우 labels Oct 17, 2024
@Dobbymin Dobbymin linked an issue Oct 17, 2024 that may be closed by this pull request
@Dobbymin Dobbymin self-assigned this Oct 17, 2024
@Dobbymin Dobbymin requested review from Diwoni and JYN523 October 19, 2024 13:22
Copy link
Collaborator

@Diwoni Diwoni Oct 19, 2024

Choose a reason for hiding this comment

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

페이지 코드가 한 눈에 잘 들어오게 훨씬 깔끔해졌네요!
고생하셨습니다!

},
onError: (error) => {
console.error('안부전화 서비스 신청에 실패했습니다.', error);
alert('안부전화 서비스 신청에 실패했습니다.');
console.log('서비스 이용 시간을 잘 못 입력하셨습니다.');
Copy link
Collaborator

Choose a reason for hiding this comment

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

alert() 로 바꾸는 게 낫지 않을까욤?
띄어쓰기도..!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 그렇네요! 수정하겠습니다!

@Dobbymin Dobbymin merged commit 3d9d35f into Weekly Oct 20, 2024
1 check passed
@Dobbymin Dobbymin deleted the Feat/issue-#79 branch October 20, 2024 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📡 API 비동기 통신 코드를 짜는 경우, 백엔드와의 통신하는 경우 ✨ Feature 새로운 기능 추가 및 구현하는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

안부전화 신청 포인트 관련 기능 구현
2 participants