Skip to content

[ZEPPELIN-2521] fix: Confusing axis description in advanced-transformation#2328

Closed
1ambda wants to merge 1 commit into
apache:masterfrom
1ambda:ZEPPELIN-2521/confusing-desc-for-axis-in-adtr
Closed

[ZEPPELIN-2521] fix: Confusing axis description in advanced-transformation#2328
1ambda wants to merge 1 commit into
apache:masterfrom
1ambda:ZEPPELIN-2521/confusing-desc-for-axis-in-adtr

Conversation

@1ambda
Copy link
Copy Markdown
Member

@1ambda 1ambda commented May 9, 2017

What is this PR for?

Fixed confusing axis descriptions in advanced-transformation.

What type of PR is it?

[Improvement]

Todos

NONE

What is the Jira issue?

ZEPPELIN-2521

How should this be tested?

  1. Install any helium visualization package written with advanced-transformation. For example ultimate-heatmap-chart.
  2. Open the setting menu.

Screenshots (if appropriate)

Before

2521_before

After

2521_after

Questions:

  • Does the licenses files need update? - NO
  • Is there breaking changes for older versions? - NO
  • Does this needs documentation? - NO

@1ambda
Copy link
Copy Markdown
Member Author

1ambda commented May 10, 2017

@soralee
Copy link
Copy Markdown
Contributor

soralee commented May 10, 2017

Let me test it out.
Thanks @1ambda for fix!

@1ambda 1ambda force-pushed the ZEPPELIN-2521/confusing-desc-for-axis-in-adtr branch from be5ebd0 to 78a39b2 Compare May 13, 2017 12:07
@1ambda 1ambda closed this May 13, 2017
@1ambda 1ambda reopened this May 13, 2017
@1ambda 1ambda closed this May 13, 2017
@1ambda 1ambda reopened this May 13, 2017
@Leemoonsoo
Copy link
Copy Markdown
Member

LGTM and merge to master if no further comments.

@soralee
Copy link
Copy Markdown
Contributor

soralee commented May 15, 2017

Tested and LGTM!

@asfgit asfgit closed this in 56cee76 May 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants