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

[FE] #14 차트 관련 통신로직 일부 구현 #52

Merged
merged 2 commits into from
Sep 5, 2023
Merged

[FE] #14 차트 관련 통신로직 일부 구현 #52

merged 2 commits into from
Sep 5, 2023

Conversation

novice1993
Copy link
Contributor

  • 단일 차트 통신 로직 구현
  • 최초 실행 시 1회 호출 후 정각 혹은 30분에 맞춰서 10분 마다 신규 API 요청 발생
  • 현재 단일 주식 로직만 구현된 상황으로, 금일 시행될 개발자 테스트 작업을 위해 작업 완료된 부분까지 merge 하려함

@novice1993
[Feat] 차트 통신 및 그래피 렌더링 구현
8f6a761 
- 서버에서 데이터 패칭 후 차트 그리는 기본 로직 구현
- 최초 렌더링 이후 정각 혹은 30분 마다 API 호출하여 데이터 갱신되도록 로직 구현
- 현재 단일 주식 로직만 구현된 상황으로, 추후 로직 확장 예정

Issues #14
@novice1993
[Feat] API 요청 주기 수정
bae4337 
- 기존 : 정각/30분에 맞춰서 단일 주가데이터 API 요청 후 30분 주기로 신규 API 요청
- 변경 : 10분 주기로 신규 API 요청
- 변경 사유 : 서버 시간 - 클라이언트 시간 불일치 혹은 API 호출 시 오차 발생으로 인한 데이터 불일치 (서버는 갱신 되었으나, 클라이언트는 갱신 X) 예방 차원에서 API 호출 주기를 더 짧게설정

- 이외 변경 사항 : Loading Indicator, Error Indicator 관련하여 CSS 일부 수정

Issues #14
@novice1993 novice1993 added the FE Front End label Sep 5, 2023
@novice1993 novice1993 self-assigned this Sep 5, 2023
@sirloinbh
Copy link
Collaborator

확인 및 approval 했습니다.
그리고 리액트 쿼리 사용 예시 잘 봤습니다.

@sinjw
Copy link
Collaborator

sinjw commented Sep 5, 2023

확인했습니다

@novice1993 novice1993 merged commit d2b5bca into dev-client Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sinjw sinjw sinjw approved these changes

@sirloinbh sirloinbh sirloinbh approved these changes

Assignees

@novice1993 novice1993

Labels
FE Front End
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@novice1993 @sirloinbh @sinjw