-
Notifications
You must be signed in to change notification settings - Fork 32
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
docs: self-managed-CRDs #660
Conversation
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 had a bunch of corrections and suggestions, please take a look 😄
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 can't believe we managed to pull this off. I can only imagine how much complexity is behind, but at the surface a lot of that is abstracted away. Awesome!
I suggested some "make-up". Please review and merge when happy.
Really excited about this one! 🎉 It opens up so many wonderful possibilities (as can be seen by how many issues I've made that reference this PR 😅) |
@llarsson @cristiklein if you are happy with the final changes I will merge this. |
@robinAwallace 🚀 from my side. |
Can/should we make it so that this only shows up once we have the possibility to roll this out with a new Apps release? |
...we do have version specifiers on the docs, these days, so I figured it's weird if this documents something that is "upcoming", unless it's clear about doing so. |
I can wait until v0.33 is released, as this should be part of v0.34. Then this will just be shown on main (which is true) and v0.34. |
0d123d6
to
7114533
Compare
7114533
to
6deb82e
Compare
🚀 |
Add public docs for self-managed CRDs, part of this PR.