Skip to content

New OpenShift service mesh section inside integrations #585

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

Merged
merged 4 commits into from
Oct 3, 2022

Conversation

josunect
Copy link
Contributor

@josunect josunect commented Sep 29, 2022

I was not sure where to place the new page.
I've created a new section, "Integrations", but it could be moved directly to a OSSMC section or inside other page.
Feedback is welcome!

https://deploy-preview-585--kiali.netlify.app/docs/integrations/ossm-console/

@josunect josunect self-assigned this Sep 29, 2022
@josunect josunect requested a review from jshaughn October 3, 2022 07:38
Copy link
Contributor

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

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

Hi @josunect, this looks good. I like the idea of an Integrations section and I think it makes sense to be at the bottom, underneath the sections that deal with standalone Kiali. I have one suggested revision for a sentence with a grammar issue. Feel free to revise my revision :-)

@josunect
Copy link
Contributor Author

josunect commented Oct 3, 2022

Hi @josunect, this looks good. I like the idea of an Integrations section and I think it makes sense to be at the bottom, underneath the sections that deal with standalone Kiali. I have one suggested revision for a sentence with a grammar issue. Feel free to revise my revision :-)

Thanks, @jshaughn ! It looks more clear and friendly :)

Copy link
Contributor

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

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

LGTM! @josunect , the plugin is released and this content is already valid, please also create a backport PR for the current branch.

Copy link
Contributor

@jshaughn jshaughn left a comment

Choose a reason for hiding this comment

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

Whoops, forgot to approve!

@josunect josunect merged commit c9ac6fb into kiali:staging Oct 3, 2022
@josunect josunect deleted the ossmc-docs branch October 3, 2022 13:33
josunect added a commit to josunect/kiali.io that referenced this pull request Oct 3, 2022
* New ossmc section

* ossmc

* Put integrations at the bottom

* Update content/en/docs/Integrations/ossm-console.md

Co-authored-by: Jay Shaughnessy <[email protected]>

Co-authored-by: Jay Shaughnessy <[email protected]>
hhovsepy added a commit that referenced this pull request Oct 4, 2022
New OpenShift service mesh section inside integrations (#585)
hhovsepy pushed a commit to hhovsepy/kiali.io that referenced this pull request Apr 5, 2024
* New ossmc section

* ossmc

* Put integrations at the bottom

* Update content/en/docs/Integrations/ossm-console.md

Co-authored-by: Jay Shaughnessy <[email protected]>

Co-authored-by: Jay Shaughnessy <[email protected]>
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