Skip to content

Updated tinymath ref links#27085

Merged
cqliu1 merged 1 commit intoelastic:masterfrom
cqliu1:fix/tinymath-links
Dec 12, 2018
Merged

Updated tinymath ref links#27085
cqliu1 merged 1 commit intoelastic:masterfrom
cqliu1:fix/tinymath-links

Conversation

@cqliu1
Copy link
Contributor

@cqliu1 cqliu1 commented Dec 12, 2018

Closes #26451.

This updates the TinyMath reference links to https://www.elastic.co/guide/en/kibana/current/canvas-tinymath-functions.html.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@w33ble w33ble left a comment

Choose a reason for hiding this comment

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

The description says it's using https://www.elastic.co/guide/en/kibana/current/canvas-tinymath-functions.html, which seems correct.

However, the code is actually linking to https://www.elastic.co/guide/en/kibana/current/canvas.html

@cqliu1 cqliu1 force-pushed the fix/tinymath-links branch from 696df7f to b1742c5 Compare December 12, 2018 22:42
@cqliu1
Copy link
Contributor Author

cqliu1 commented Dec 12, 2018

Oops, I pasted the wrong link. Using the correct links now.

Copy link
Contributor

@w33ble w33ble left a comment

Choose a reason for hiding this comment

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

:shipit:

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cqliu1 cqliu1 merged commit af851a3 into elastic:master Dec 12, 2018
@cqliu1 cqliu1 deleted the fix/tinymath-links branch December 12, 2018 23:19
cqliu1 added a commit to cqliu1/kibana that referenced this pull request Dec 12, 2018
cqliu1 added a commit to cqliu1/kibana that referenced this pull request Dec 12, 2018
cqliu1 added a commit that referenced this pull request Dec 13, 2018
cqliu1 added a commit that referenced this pull request Dec 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants