Skip to content

Conversation

kyoji63
Copy link

@kyoji63 kyoji63 commented Sep 10, 2025

What this PR does / why we need it:
Fixed documentation mistakes from version 3.17.0 and later.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Follow up for: litmuschaos/litmus#5111

Special notes for your reviewer:

Checklist:

follow up for: litmuschaos/litmus#5111
Fixed documentation mistakes from version 3.17.0 and later.

Signed-off-by: kyoji63 <[email protected]>
Copy link

vercel bot commented Sep 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
litmus-docs Ready Ready Preview Comment Sep 10, 2025 10:20am

Copy link

netlify bot commented Sep 10, 2025

Deploy Preview for docs-litmuschaos ready!

Name Link
🔨 Latest commit b66b902
🔍 Latest deploy log https://app.netlify.com/projects/docs-litmuschaos/deploys/68c14f21c36c2500087f81c4
😎 Deploy Preview https://deploy-preview-336--docs-litmuschaos.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@PriteshKiri
Copy link
Contributor

Hey @kyoji63,

Thanks for raising the PR. Unfortunately, this issue has already been fixed with #335

@Jonsy13
Copy link
Contributor

Jonsy13 commented Sep 16, 2025

Closing this PR as it has been already fixed. Thanks for the effort @kyoji63

@Jonsy13 Jonsy13 closed this Sep 16, 2025
@kyoji63
Copy link
Author

kyoji63 commented Sep 17, 2025

@PriteshKiri @Jonsy13
Thank you for your comment.
I reviewed the changes in #335, but I noticed that the following part in dex-deployment.yaml has not been fully addressed.
This PR aims to cover that remaining fix.

Could you please review it again?

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.

3 participants