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.

Sign up for GitHub

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

Jump to bottom

[ST-2] TodoManager.create()에서 NotiManager.notify() 호출하도록 수정 #4

Open

iamapark wants to merge 2 commits into master from st-2_Todomanager_notify_연동_구현_박진영

iamapark commented Feb 21, 2018 •

edited

Loading

관련 문서: https://www.slipp.net/wiki/pages/viewpage.action?pageId=28279379


          [ST-2] TodoManager.create()에서 NotiManager.notify() 호출하도록 수정

a68a8dd

iamapark requested a review from dhrim

February 21, 2018 12:38


          [ST-2] NotiManager.notify() 호출 counter 코드 위치 수정

5643fc3

- exception 체크 전에 카운터를 +1 하도록 수정했습니다. 기존 코드는 exception이 발생하는 경우 카운터를 +1하지 않았기 때문에 메소드 호출 횟수 검증이 제대로 되지 않고 있었습니다

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet