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

[#1269] Chart > Legend 영역 통계 > null 처리 #1270

Merged
merged 1 commit into from
Aug 23, 2022
Merged

[#1269] Chart > Legend 영역 통계 > null 처리 #1270

merged 1 commit into from
Aug 23, 2022

Conversation

jhee564
Copy link
Collaborator

@jhee564 jhee564 commented Aug 23, 2022

처리 내용

  • 합계, 최솟값, 최댓값, 평균 계산시 null과 undefined는 제외하도록 함
  • 마지막값 계산 시 null과 undefined는 'Null'로 표시되도록함
  • 계산된 값이 javascript기준 MAX_SAFE_INTEGER을 초과했거나 MIN_SAFE_INTEGER 미만일 경우 console warning이 출력되도록함

예시

legend_aggregation_table_null

@jhee564
[#1269] Chart > Legend 영역 통계 > null 처리
6dda85b 
#####################################
- 합계, 최솟값, 최댓값, 평균 계산시 null과 undefined는 제외하도록 함
- 마지막값 계산 시 null과 undefined는 'Null'로 표시되도록함
- 계산된 값이 javascript기준 MAX_SAFE_INTEGER을 초과했거나 MIN_SAFE_INTEGER 미만일 경우 console warning이 출력되도록 함
@jhee564 jhee564 merged commit f17a2d3 into 3.0 Aug 23, 2022
@jhee564 jhee564 deleted the 1269_chart_legend_null branch August 23, 2022 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kdeun1 kdeun1 kdeun1 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jhee564 @kdeun1