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.

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

Bug/#341 유저가 다른 주식에 접근할 때 예전 주식의 실시간 데이터가 보이는 버그 수정 #350

Merged
merged 14 commits into from
Dec 4, 2024
Merged

Bug/#341 유저가 다른 주식에 접근할 때 예전 주식의 실시간 데이터가 보이는 버그 수정 #350

merged 14 commits into from
Dec 4, 2024

Conversation

swkim12345
Copy link
Collaborator

close #

✅ 작업 내용

  • 유저가 다른 주식에 들어갔을 때 다른 주식방의 실시간 데이터가 보이는 이슈 수정

😎 체크 사항

  • label 설정 확인
  • 브랜치 방향 확인

@swkim12345
♻️ refactor: 분봉 데이터 callback 형태로 바꿔 우선순위 큐 적용, 리팩토링
bbb9b62
@swkim12345
♻️ refactor: 분단위 데이터 수집 stock limit 200, 콜백함수로 리팩토링
fc145a9
@swkim12345
🐛 fix: afterUpdate 적용 위한 upsert구문으로 변경
fc840c9
@swkim12345
💄 style: 분단위 테스트 이후 테스트 코드 삭제 및 조건 원복
9ae9f3a
@swkim12345
💄 style: dto 안 쓰이는 속성 삭제
3ae677c
@swkim12345
💄 style: console.log 삭제
31d2ef7
@swkim12345
📝 docs: liveData에 unsubscribe, subscribe 메시지 info로 출력 추가
72edf0d
@swkim12345
🐛 fix: 알람을 한번만 보내고 삭제처리하게 만듦
65ecaaa
@swkim12345
🐛 fix: 유저가 동시에 다른 주식 방에 있는 것을 방지
ac48ade
@swkim12345
🐛 fix: 유저가 동시에 다른 주식에 접속하는 것을 막음
c69ebd9
@swkim12345
🐛 fix: client leave 추ㅏㄱ
d45d2e6
@swkim12345
🐛 fix: leave 코드가 values를 검사하는 로직보다 뒤에 있어 발생되는 문제 해결
a4571c1
@swkim12345
🐛 fix: delete도 이동
f2ad68b
@swkim12345
🐛 fix: deadlock 발생 없이 동시성 제어
629b47d
@swkim12345 swkim12345 added the 🐞 bugfix Something isn't working label Dec 4, 2024
baegyeong
baegyeong approved these changes Dec 4, 2024
View reviewed changes
Copy link
Collaborator

@baegyeong baegyeong left a comment

Choose a reason for hiding this comment

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

일단 서버에서 되는지 확인하고 프론트도 적용할게요

xjfcnfw3
xjfcnfw3 approved these changes Dec 4, 2024
View reviewed changes
Copy link
Collaborator

@xjfcnfw3 xjfcnfw3 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍 👍

packages/backend/src/scraper/openapi/api/openapiMinuteData.api.ts
.addSelect('COUNT(alarm.id)', 'alarmCount')
.groupBy('stock.id')
.orderBy('alarmCount', 'DESC')
.limit(this.STOCK_LIMITS)
Copy link
Collaborator

Choose a reason for hiding this comment

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

이렇게 하면 일부는 알림을 못보내지 않나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이거 없앤 다음에 다시 보낼게요!

@xjfcnfw3 xjfcnfw3 merged commit c4d1d9e into dev-be Dec 4, 2024
@baegyeong baegyeong linked an issue Dec 4, 2024 that may be closed by this pull request
[BUG] - 주식 창을 옮겨도 이전 라이브데이터 소켓이 연결된 버그 #349
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baegyeong baegyeong baegyeong approved these changes

@xjfcnfw3 xjfcnfw3 xjfcnfw3 approved these changes

Assignees

@swkim12345 swkim12345

Labels
🐞 bugfix Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] - 주식 창을 옮겨도 이전 라이브데이터 소켓이 연결된 버그
3 participants
@swkim12345 @xjfcnfw3 @baegyeong