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

[Refactor/recommendation 223] 탑승/하차 코드 리팩토링 #232

Open
wants to merge 4 commits into
base: staging
Choose a base branch
from

Conversation

yunyoung1819
Copy link
Contributor

💡 개요

📑 작업 사항

  • 탑승/하차 코드 리팩토링
  • TODO 주석 부분 1개 수정

✒️ 코드 리뷰 요청 사항

✔️ 코드 리뷰 반영 사항

Copy link
Member

@devsungmin devsungmin left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 👍🏼

webtoonStatusCount,
status
);
return antCoinService.joinWebtoon(userId, webtoonId, response, status);
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
return antCoinService.joinWebtoon(userId, webtoonId, response, status);
return antCoinService.joinWebtoon(joinWebtoonDto);

이런식으로 dto 객체를 만들어서 하는 건 어떠신가요??

Copy link
Contributor 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
refactor 예쁘게 다듬어~
Projects
None yet
Development

Successfully merging this pull request may close these issues.

탑승(Recommendation) 도메인 수정 및 로직 개선
2 participants