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

Sorting function of data table does not work properly. #973

Closed
Taehui opened this Issue Nov 30, 2018 · 3 comments

Comments

Projects
None yet
4 participants
@Taehui
Copy link
Collaborator

Taehui commented Nov 30, 2018

Describe the bug

  • 테이블의 x축을 데이터 정렬하고자 하는 경우 동작 안함
  • 테이블차트 복수의 데이터 정렬 안됨
    --> X, Y축 무관 A컬럼 정렬하고 B컬럼 정렬 추가하면 A컬럼 정렬이 풀리는 증상

Expected behavior

  • 정렬이 정상적으로 동작해야 함

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

@Taehui Taehui added the bug label Nov 30, 2018

@YooYoungmo YooYoungmo self-assigned this Dec 14, 2018

@YooYoungmo

This comment has been minimized.

Copy link
Collaborator

YooYoungmo commented Dec 14, 2018

X 축 데이터 정렬 문제

문제 재현

ShipMode를 정렬을 변경해도 테이블에 보이는 데이터가 변함 없음
image

원인

메타트론 API를 호출시 X 축에 대한 정렬값을 정상적으로 보내고 있으나, 메타트론에서 엔진(드루이드)에 조회시 X 축에 대한 정렬이 ascending으로 고정되어 있음

@YooYoungmo

This comment has been minimized.

Copy link
Collaborator

YooYoungmo commented Dec 14, 2018

먼저 X축에 데이터 정렬 오류 진행 합니다.

@YooYoungmo

This comment has been minimized.

Copy link
Collaborator

YooYoungmo commented Dec 17, 2018

테이블차트 복수의 데이터 정렬 안됨

문제 재현

image

원인

  • 메타트론에서 엔진(드루이드)에 조회시 화면에서 설정한 것 처럼 Y 축의 정렬 순서(City, State)로 요청해야 원하는 결과를 얻을 수 있음.
  • 테스트 결과 만약 사용자가 화면에서 State 정렬을 변경하게 되면 메타트론 서버에서 State, City 순서로 드루이드에 요청하기 때문에 사용자가 원하는 순서 데이터가 조회되지 않음.
  • 결과적으로는 사용자가 화면에서 설정한 순서대로 드루이드에 정렬 요청을 해야함.

YooYoungmo added a commit to YooYoungmo/metatron-discovery that referenced this issue Dec 18, 2018

YooYoungmo added a commit to YooYoungmo/metatron-discovery that referenced this issue Dec 19, 2018

YooYoungmo added a commit to YooYoungmo/metatron-discovery that referenced this issue Dec 19, 2018

@kyungtaak kyungtaak added the @chart label Dec 30, 2018

@kyungtaak kyungtaak added this to the 3.2.0 milestone Dec 30, 2018

@kyungtaak kyungtaak closed this Jan 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment