-
Notifications
You must be signed in to change notification settings - Fork 27
doc: quick fix to use the new client-certificates relation #1227
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
doc: quick fix to use the new client-certificates relation #1227
Conversation
I don't think I have access to add a label. |
|
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.
Hi @nobuto-m, thank you very much for your contribution to the docs. The issue here is that this is being merged into the main branch (which corresponds to the 14/edge track) instead of the 16/edge branch.
Try merging into 16/edge - I think some of the fixes you made to the tutorial are already in the docs for 16.
Our docs versioning is rather new, so this can still be a bit confusing. I've created an issue (#1228) for us to update the contribution guide to include documentation details.
|
@nobuto-m Regarding the broken terraform link: this is a known issue due to their CDN rejecting GitHub runners/link checkers. This will be addressed separately and is not a blocker for merging this PR. |
8c961b9 to
cbaa923
Compare
This is very confusing. Most of the time, I assume development is happening in the main/master branch of any projects and stable releases like |
a-velasco
left a comment
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.
Thank you!
Issue
Closes: #1225
Solution
Checklist