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

#1095 Must be able to download over 1000 original data #1140

Merged
merged 2 commits into from
Dec 20, 2018

Conversation

eltriny
Copy link
Contributor

@eltriny eltriny commented Dec 19, 2018

Description

현재 대시보드 내 차트 위젯안에서 원본데이터를 다운로드 받을 수 있는 기능이 있지만, 현재 1000건에 대해서만 다운로드 받을 수 있음. 현재 대시보드 내 데이터소스 (원본) 다운로드는 10만건을 다운로드 받을 수 있음으로, 차트 위젯 내 원본데이터도 10만건은 다운로드 받을 수 있어야 함

Related Issue : #1095

How Has This Been Tested?

  1. 10만건 이상 되는 데이터소스를 이용하여 대시보드 및 차트를 생성합니다.
  2. 차트 다운로드에서 데이터가 10만건까지 다운로드 되는지 확인합니다.
  3. 데이터 다운로드 시 하단의 안내 문구가 10만건을 넘어갈 경우에만 표시되는지 확인합니다.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project. it will be added soon
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document. it will be added soon
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Additional Context

@eltriny eltriny added the @dashboard Component Name : Dashboard / Chart label Dec 19, 2018
@ufoscw ufoscw added the testbed3 Review Test Server 3 label Dec 20, 2018
@alchan-lee
Copy link
Contributor

@eltriny
기획분들과 다시 이야기되어서 100만 건으로 변경했습니다.
커밋은 제가 해두었는데 내용 공유차 말씀드립니다~!

@alchan-lee

This comment has been minimized.

1 similar comment
@alchan-lee
Copy link
Contributor

run build
deploy to 3

@kyungtaak kyungtaak merged commit 87063f8 into 3.1.1 Dec 20, 2018
@alchan-lee alchan-lee deleted the f#1095-download_over_1000_original_data branch January 7, 2019 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@dashboard Component Name : Dashboard / Chart testbed3 Review Test Server 3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants