Skip to content

Added blog entry describing how to connect e.g. Tableau to Apache Kylin#862

Merged
shaofengshi merged 1 commit intoapache:documentfrom
pnaszarkowski:document
Sep 27, 2019
Merged

Added blog entry describing how to connect e.g. Tableau to Apache Kylin#862
shaofengshi merged 1 commit intoapache:documentfrom
pnaszarkowski:document

Conversation

@pnaszarkowski
Copy link

…t Tableau with Kylin

@asf-ci
Copy link

asf-ci commented Sep 26, 2019

Can one of the admins verify this patch?

@pnaszarkowski pnaszarkowski marked this pull request as ready for review September 26, 2019 19:43
@pnaszarkowski pnaszarkowski changed the title Added blog entry 2019-09-22-kylin-tableau.md describing how to connec… Added blog entry describing how to connect e.g. Tableau to Apache Kylin Sep 26, 2019
@pnaszarkowski
Copy link
Author

To be reviewed by @shaofengshi ?

Copy link
Contributor

@shaofengshi shaofengshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@shaofengshi shaofengshi merged commit 7767fe3 into apache:document Sep 27, 2019
@pnaszarkowski
Copy link
Author

@shaofengshi I see some formatting issues on the blog after publishing (https://kylin.apache.org/blog/2019/09/22/kylin-tableau/), please take a look and let me know if that should be fixed in the blog entry or website engine?

  • 3rd and 4th level sections' titles are not rendered correctly
    image

  • numbering is broken when numbered points contain images
    image

@shaofengshi
Copy link
Contributor

In my local preview with Jekyll, they appeared good; anyway, I inserted a blank line there to see whether it has any change.

屏幕快照 2019-09-29 下午9 35 20

屏幕快照 2019-09-29 下午9 46 41

@pnaszarkowski pnaszarkowski deleted the document branch October 1, 2019 19:40
@pnaszarkowski
Copy link
Author

@shaofengshi @nichunen - thanks for fixing this, blog entry looks OK now :)

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