Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Fixed broken hyperlinks in carbon-graphs README #203

Merged
merged 4 commits into from
Aug 13, 2021
Merged

Fixed broken hyperlinks in carbon-graphs README #203

merged 4 commits into from
Aug 13, 2021

Conversation

sdadn
Copy link
Contributor

@sdadn sdadn commented Aug 3, 2021

Summary

Closes #198

Deployment Link

https://terra-graphs-deployed-pr-#.herokuapp.com/

Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra
@cerner/carbon

@sdadn sdadn added documentation Improvements or additions to documentation 📦 carbon-graphs labels Aug 3, 2021
@sdadn sdadn self-assigned this Aug 3, 2021
@mjhenkes mjhenkes temporarily deployed to terra-graphs-issue-198-5mfqtrt August 3, 2021 17:03 Inactive
- [Bubble](../terra-docs/docs/controls/Bubble.md#usage)
- [Bubble Single Dataset](../terra-docs/docs/controls/BubbleSingleDataset.md)
- [Bubble Multiple Dataset](../terra-docs/docs/controls/BubbleMultipleDataset.md)
- [Line](../terra-graphs-docs/docs/controls/Line.md#usage)
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like these links are not working as expected.
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They should be working on GitHub. If you select Rich Text Difference and try the links, they should work.

CleanShot 2021-08-04 at 20 18 08@2x

Copy link
Contributor

Choose a reason for hiding this comment

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

Even in local, it dosen't look good
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is also happening in other .md files that weren't updated so I think it's a rendering issue. Either way, it's beyond the scope of this story.

@mjhenkes mjhenkes temporarily deployed to terra-graphs-issue-198-5mfqtrt August 5, 2021 01:22 Inactive
@mjhenkes mjhenkes temporarily deployed to terra-graphs-issue-198-5mfqtrt August 6, 2021 14:12 Inactive
@mjhenkes mjhenkes temporarily deployed to terra-graphs-issue-198-5mfqtrt August 13, 2021 21:54 Inactive
@sdadn sdadn merged commit 89c8126 into main Aug 13, 2021
@sdadn sdadn deleted the issue-198 branch August 13, 2021 22:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation 📦 carbon-graphs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken hyperlinks on the carbon-docs package
5 participants