-
Notifications
You must be signed in to change notification settings - Fork 879
docs: convert alerts to use GitHub Flavored Markdown (GFM) #16850
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
Changes from 4 commits
ba56dad
40159cf
50d4d50
dc9f67b
ced07b4
1eaeab8
48cf62b
ca70325
0dab25c
af0fdd8
6b4d36a
06c8b36
1735d16
e527a53
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||
---|---|---|---|---|---|---|---|---|
|
@@ -172,12 +172,9 @@ more. | |||||||
|
||||||||
## Browser-only connections | ||||||||
|
||||||||
<blockquote class="info"> | ||||||||
|
||||||||
Browser-only connections is an Enterprise and Premium feature. | ||||||||
[Learn more](https://coder.com/pricing#compare-plans). | ||||||||
|
||||||||
</blockquote> | ||||||||
> [!NOTE] | ||||||||
> Browser-only connections is an Enterprise and Premium feature. | ||||||||
> [Learn more](https://coder.com/pricing#compare-plans). | ||||||||
|
||||||||
Some Coder deployments require that all access is through the browser to comply | ||||||||
with security policies. In these cases, pass the `--browser-only` flag to | ||||||||
|
@@ -189,12 +186,9 @@ via the web terminal and | |||||||
|
||||||||
### Workspace Proxies | ||||||||
|
||||||||
<blockquote class="info"> | ||||||||
|
||||||||
Workspace proxies are an Enterprise and Premium feature. | ||||||||
[Learn more](https://coder.com/pricing#compare-plans). | ||||||||
|
||||||||
</blockquote> | ||||||||
> [!NOTE] | ||||||||
> Workspace proxies are an Enterprise and Premium feature. | ||||||||
> [Learn more](https://coder.com/pricing#compare-plans). | ||||||||
Comment on lines
+191
to
+193
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Instead use label on the heading. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. leaving out of this PR per #16850, slack discussion, and other comments - the way the site renders the anchor URL with Alternatively, we can consider making the whole section something like "Everything from here down requires a license..." |
||||||||
|
||||||||
Workspace proxies are a Coder Enterprise feature that allows you to provide | ||||||||
low-latency browser experiences for geo-distributed teams. | ||||||||
|
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.
❤️
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.
it doesn't like this one for some reason 🤷
I'm playing with the indentation locally
current state:
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.
I have a hunch it's because this one is nested in an ordered list.
Was this fixed by ced07b4? If not, does it hurt the flow/semantics if we move the blockquote to be below the numbered list?