Conversation
잘못 적용된 bid와 ask 처리 코드를 수정했습니다.
- ticker별로 호출하도록 변경 issue: #39
…nto feat/realitybot
- 복수 종목 vwap 계산 중 인스턴스 문제 발생 issue : #39
issue: #62
- ticksQueue - lastSequentialId issue: #62
주문 받을 때에는 db에 있는 종목을 기준으로 요청을 검증하고, 내부적으로 호출될 때 ticker에 해당하는 주문큐, 호가목록이 없을 경우 생성됩니다.
…nto feat/chartdata # Conflicts: # src/main/java/com/cleanengine/coin/chart/controller/ChartDataController.java
서버가 시작될 때, db상에 icon이 초기화가 안되었다면, static resource 기반으로 db상에 icon을 업데이트합니다. AssetController의 응답형식에도 icon을 추가했습니다. icon은 SVG가 base64 인코딩된 문자열로 반환됩니다.
config: LOG_LEVEL 환경변수를 통해 로그 레벨을 동적으로 변경할 수 있도록 변경
이후 종목 추가 API에서 svg 파일을 encoded bytes로 변환하는 메서드를 필요로 할 수 있기 때문에 접근자를 public으로 변경하고 이름을 더 이해하기 쉽도록 변경했습니다.
config: root 기본 레벨을 INFO로 추가
종목 API에 icon 반환 추가, 주문, 호가 종목추가 대응로직 추가
Feat: chartdata
WorkingServerProfile을 ChartDataController에 적용시, bean 초기화 오류가 발생해 일시적으로 허용했습니다. 통합 테스트 전에 Scheduling 로직이 비활성화 될 수 있도록 재수정 필요할 것 같습니다.
fix: 일시적으로 WorkingServerProfile 비활성화
주문 복원 기능이 체결엔진및 주문 자동 생성보다 우선시 실행되어야 하기 때문에, Order 설정을 수정했습니다.
dev, prod 환경에서 서버가 재시작 되었을 때, DB상의 미완료된 주문들을 Queue에 복원하는 기능을 추가했습니다.
config: sonarqube 설정 추가
* test: 체결 통합 테스트 프로필 설정 추가 주문 생성 봇의 로직이 포함되지 않은 환경에서 수행되도록 profile 설정을 변경했습니다. * fix: testcontainers test 전체 테스트 대상에서 제외 기존 Disabled Annotation만으로는 sub클래스 테스트 제외까지 시키지 못했던점을 고려해 Tag 기반으로 전역 테스트 제외 설정을 했습니다. * config: 개발환경 편의성을 위한 jacoco 설정변경 개발 환경에서 자신의 패키지에 대해서만 테스트 커버리지를 생성할 수 있도록 주석처리된 설정을 반영했습니다 * fix: 임시로 websocket 테스트 비활성화 jacoco 설정 반영을 위해 실패하던 테스트를 임시로 비활성화해두었습니다. 다른 branch에서 수정예정입니다.
검증을 추가해서 NUll로 들어온다면 NPE를 발생시키게 수정했습니다.
Config/ci cd
[BE] 차트데이터의 서비스 로직의 모든 단위테스트 및 리펙토링 추가
|
BHyeonKim
approved these changes
Jun 4, 2025
Contributor
BHyeonKim
left a comment
BHyeonKim
left a comment
There was a problem hiding this comment.
Main에 병합하고 이슈는 이후에 해결하도록 해요.
Contributor
|
메인 병합할 내용 확인했습니다. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
✨ 작업내용
dev->main운영 반영 건입니다.