-
Notifications
You must be signed in to change notification settings - Fork 683
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Docs] Fix links in Basic Concepts #1158
[Docs] Fix links in Basic Concepts #1158
Conversation
This pull request is automatically deployed with Now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is probably good to merge, just having issues generating rendered content locally.
|
||
For more information, see [Container extensibility]. | ||
For more information, see [Container extensibility][]. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't rendering correctly in Markdown preview, and I'm having issues getting npm run start
to run locally. Can you confirm this looks okay in rendered output (and maybe help me resolve my local error 😄 )
Optimizing the critical path is important to get the best performance out of a progressive web application. | ||
<sub>[ _[Help write this topic][critical path]_ ]</sub> | ||
|
||
[container extensibility]: {{ site.baseurl }}{%link technologies/basic-concepts/container-extensibility/index.md %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This also doesn't render well, but I'm suspecting is just a markdown preview issue. Help me with my local, and I presume this is good to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jcalcaben helped me get ruby dependencies installed, and rendered output looks perfect. Approved!
Description
Fix links in Basic Concepts topic to just point to the project repo instead of random issues.
Related Issue
Closes #1157
Checklist: