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

에러 로그를 추가하고 일부 버그를 수정합니다. #44

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

litsynp
Copy link
Contributor

@litsynp litsynp commented Feb 5, 2024

에러 로그를 추가하고 일부 버그를 수정합니다.

  • AppError 생성자에 AppError를 넣은 이슈
  • RegisterUser 간 잘못된 에러 반환하는 이슈
  • page / size가 0 이하일 때 panic이 일어나는 이슈

로깅 라이브러리는 zerolog를 사용합니다.

@litsynp litsynp requested a review from barabobBOB February 5, 2024 13:47
@litsynp litsynp self-assigned this Feb 5, 2024
@litsynp litsynp force-pushed the chore/log-minimum branch 2 times, most recently from 92f20f4 to 0eb5b77 Compare February 5, 2024 14:05
fix: runtime and compile errors
Copy link
Member

@barabobBOB barabobBOB left a comment

Choose a reason for hiding this comment

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

확인했습니다 :) 수고하셨습니다 !

@litsynp
Copy link
Contributor Author

litsynp commented Feb 6, 2024

🙇

@litsynp litsynp merged commit 5d53bf3 into main Feb 6, 2024
1 check passed
@litsynp litsynp deleted the chore/log-minimum branch February 6, 2024 13:51
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.

2 participants