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

version 2.8.1 #208

Merged
merged 1 commit into from
Aug 5, 2024
Merged

version 2.8.1 #208

merged 1 commit into from
Aug 5, 2024

Conversation

zbqmgldjfh
Copy link
Member

TODO

  • fix(AlertCommandApi): 사용하지 않던 API 삭제
  • chore(AdminCommandApiV2#createAlert): API 설명 추가
  • fix(AlertService): addSchedule에 람다로 넘긴 함수에 transaction이 적용되지 않던 문제를 transactionTemplate 사용하여 해결
  • test(AlertServiceTest): thread.sleep()이 아닌, await() 를 사용하도록 변경

* fix(AlertCommandApi): 사용하지 않던 API 삭제

* chore(AdminCommandApiV2#createAlert): API 설명 추가

* fix(AlertService): addSchedule에 람다로 넘긴 함수에 transaction이 적용되지 않던 문제를 transactionTemplate 사용하여 해결

* test(AlertServiceTest): thread.sleep()이 아닌, await() 를 사용하도록 변경

* chore: 발생하지 않는 예외 제거
@zbqmgldjfh zbqmgldjfh added the 🚀 Deploy Production deploy production server label Aug 5, 2024
@zbqmgldjfh zbqmgldjfh self-assigned this Aug 5, 2024
Copy link

sonarqubecloud bot commented Aug 5, 2024

Copy link

github-actions bot commented Aug 5, 2024

Unit Test Results

  48 files  ±0    48 suites  ±0   43s ⏱️ -1s
256 tests +1  246 ✔️ +1  10 💤 ±0  0 ±0 
259 runs  +1  249 ✔️ +1  10 💤 ±0  0 ±0 

Results for commit ebfa8cf. ± Comparison against base commit b65837d.

@zbqmgldjfh zbqmgldjfh merged commit b588b51 into main Aug 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Deploy Production deploy production server
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant