Skip to content

Add link to YouTube Channel, Add link to C# GitHub Template#9900

Merged
cassiebreviu merged 13 commits intomicrosoft:gh-pagesfrom
cassiebreviu:gh-pages
Jan 10, 2022
Merged

Add link to YouTube Channel, Add link to C# GitHub Template#9900
cassiebreviu merged 13 commits intomicrosoft:gh-pagesfrom
cassiebreviu:gh-pages

Conversation

@cassiebreviu
Copy link
Copy Markdown
Contributor

Added button links for the YouTube channel and to the C# doc template on the docs home page.
Staging link: https://cassiebreviu.github.io/onnxruntime/docs/

@cassiebreviu cassiebreviu requested a review from natke December 1, 2021 22:23
docs/index.md Outdated

---

## ONNX Runtime YouTube Channel
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This might get a bit lost on this page. Wondering if we should add to the links across the top?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I was wondering if that was a good spot or not. So you think putting the links above the video would be better?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I also considered just adding it to the "How to use ONNX Runtime (ORT)" table instead of it being its own section. Thoughts?

@cassiebreviu cassiebreviu requested a review from natke December 3, 2021 22:00
@natke natke closed this Jan 8, 2022
@natke natke reopened this Jan 8, 2022
@cassiebreviu cassiebreviu merged commit 30c1a3b into microsoft:gh-pages Jan 10, 2022
@cassiebreviu cassiebreviu deleted the gh-pages branch January 10, 2022 23:39
@cassiebreviu cassiebreviu restored the gh-pages branch January 10, 2022 23:39
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.

2 participants