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

Combination chart requires secondary axis settings #175

Closed
AnnieHwang opened this issue Sep 12, 2018 · 6 comments
Closed

Combination chart requires secondary axis settings #175

AnnieHwang opened this issue Sep 12, 2018 · 6 comments
Assignees
@kyungtaak @eltriny @dave-taeho
Labels
@chart Chart Component and Widget enhancement Request Change and Feature Enhancement p1 Priority 1
Milestone
3.2.0

Comments

@AnnieHwang
Copy link
Contributor

Is your feature request related to a problem? Please describe.
콤바인 차트의 보조축에 설정을 할 수 있어야 함

Describe alternatives you've considered
idcube-12-01

Additional context
Add any other context or screenshots about the feature request here.
@AnnieHwang AnnieHwang added the @dashboard Component Name : Dashboard / Chart label Sep 12, 2018
@kyungtaak kyungtaak added the enhancement Request Change and Feature Enhancement label Sep 12, 2018
@kyungtaak kyungtaak added this to the 3.1.0 milestone Sep 12, 2018
@eltriny eltriny added @chart Chart Component and Widget and removed @dashboard Component Name : Dashboard / Chart labels Oct 10, 2018
@dave-taeho
Copy link
Contributor

#175-combination_chart_secondary_axis 브랜치 추가하였습니다.

@dave-taeho
Copy link
Contributor

@kyungtaak
컴바인차트에 ChartAxis 타입의 보조축 필드 추가 요청드립니다.

2018-10-16 3 39 18

@kyungtaak
Copy link
Contributor

@dave-taeho "secondaryAxis" 라는 속성명으로 추가 했습니다. (그리고.. 브랜치 명명 규칙이 잘못되었네요.)

dave-taeho added a commit that referenced this issue Nov 20, 2018
@dave-taeho
#175 Combine chart secondary axis option panel
23ee89e
@babokim
Copy link
Contributor

babokim commented Nov 26, 2018

@kyungtaak 이 이슈에서 각 measure에 대해 primary, secondary axis 사용하는 것을 설정하는 기능이 포함되는 건가요?

@kyungtaak kyungtaak added the p1 Priority 1 label Nov 26, 2018
@esezin
Copy link

esezin commented Nov 29, 2018

@babokim @kyungtaak
보조축 옵션에 대한 구현과 함께, 아래의 정책도 함께 포함될 예정입니다.
유형은 각 Measure의 Contextual Menu에서 설정하며, Bar Chart는 Primary Axis, Line Chart는 Secondary Axis를 사용하는 것이 현재의 정책입니다.
175-01

@kyungtaak kyungtaak modified the milestones: 3.1.0, 3.2.0 Nov 30, 2018
@dave-taeho dave-taeho mentioned this issue Dec 10, 2018
Closed
dave-taeho added a commit that referenced this issue Dec 10, 2018
@dave-taeho
#175 Combine chart => apply secondary axis
9c0636c
dave-taeho added a commit that referenced this issue Dec 11, 2018
@dave-taeho
#175 Combine chart => apply secondary axis title & label
25e1ca6
dave-taeho added a commit that referenced this issue Dec 13, 2018
@dave-taeho
Merge branch '3.1.1' into #175-combination_chart_secondary_axis
3f321d8 
# Conflicts:
#	discovery-frontend/src/app/page/chart-style/common-option.component.ts
#	discovery-frontend/src/app/page/page-share.module.ts
#	discovery-frontend/src/app/page/page.component.ts
#	discovery-server/src/main/java/app/metatron/discovery/domain/workbook/configurations/chart/CombineChart.java
#	discovery-server/src/test/java/app/metatron/discovery/domain/workbook/configurations/chart/CombineChartTest.java
dave-taeho added a commit that referenced this issue Dec 13, 2018
@dave-taeho
#175 Combine chart => fix secondary axis option error
314b3c9
@dave-taeho dave-taeho mentioned this issue Dec 13, 2018
Closed
9 tasks
dave-taeho added a commit that referenced this issue Dec 18, 2018
@dave-taeho
#175 Combine chart => add show sub-axis
7e9a9ec
@dave-taeho dave-taeho mentioned this issue Dec 18, 2018
Merged
9 tasks
dave-taeho added a commit that referenced this issue Dec 19, 2018
@dave-taeho
#175 Combine chart => fix label error & label option
f663d56
alchan-lee pushed a commit that referenced this issue Dec 19, 2018
@dave-taeho @alchan-lee
#175 combination chart secondary axis (#1106)
f749cb5 
* #516 add specification for secondary axis

* #175 Combine chart secondary axis option panel

* #175 Combine chart => apply secondary axis

* #175 Combine chart => apply secondary axis title & label

* #175 Combine chart => fix secondary axis option error

* #175 Combine chart => add show sub-axis

* #175 Combine chart => fix label error & label option
@alchan-lee
Copy link
Contributor

fixed in #1106

dave-taeho added a commit that referenced this issue Dec 19, 2018
@dave-taeho
#175 Combine chart => fix secondary axis range error
5de4392
@dave-taeho dave-taeho mentioned this issue Dec 19, 2018
Merged
9 tasks
kyungtaak pushed a commit that referenced this issue Dec 20, 2018
@dave-taeho @kyungtaak
#175 fix secondary axis range error in combine chart
343da8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kyungtaak kyungtaak

@eltriny eltriny

@dave-taeho dave-taeho

Labels
@chart Chart Component and Widget enhancement Request Change and Feature Enhancement p1 Priority 1
Projects
None yet
Milestone
3.2.0
Development

No branches or pull requests
7 participants
@babokim @kyungtaak @eltriny @AnnieHwang @dave-taeho @esezin @alchan-lee