Skip to content
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

organize and add ops guides #820

Closed
wants to merge 1 commit into from
Closed

organize and add ops guides #820

wants to merge 1 commit into from

Conversation

qrkourier
Copy link
Member

No description provided.

Copy link

vercel bot commented Mar 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated (UTC)
ziti-doc ⬜️ Ignored (Inspect) Visit Preview May 1, 2024 5:22pm

@qrkourier
Copy link
Member Author

Copying feedback from #814 because aggregating changes in this branch

@dovholuknf wrote:

On the whole, there feels like a fair bit of reference-type-words mixed throughout that probably don't belong in a guide-type document. Would you go through and find anything that feels reference-ish and unecessary and remove? IPC Agent is not necessary to understand, SNI and ALPN seem to be reference-related not guide-related to me. There might be other stuff that could be trimmed out as well but as I read through those sorts of things are striking me as not really being inline with what a 'guide' is. some bit of it, is necessary but this should focus more bringing it up and how, not the details under it. it does make me feel like there's some kind of related reference material here that could be exctracted and moved to reference

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.

None yet

2 participants