-
Notifications
You must be signed in to change notification settings - Fork 118
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: adapt landing page with better fitting titles and links #2336
Conversation
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
✅ Deploy Preview for keptn-lifecycle-toolkit ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
Signed-off-by: Moritz Wiesinger <moritz.wiesinger@dynatrace.com>
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 think we should pare down some rubbish from this page. I would delete the almost year-old video and all the text that follows it. So we just have the three ugly little icons, each of which links to the appropriate sub-section of https://main.lifecycle.keptn.sh/docs/intro/ , which has brief but intelligent descriptions of the three components or use-cases or features or whatever we are calling them.
Then each of those subsections links to the relevant Getting Started exercise and guide chapter.
Kudos, SonarCloud Quality Gate passed! |
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.
This is a big improvement so I'm approving it. I still think that the syntactical/grammatical mess in the second section about the use-cases/features should be fixed sooner rather than later, that the video should be removed, and that we should use the same names for use-cases/components/features on the landing page as we use in the docs but we can fix that later.
This PR
Notes for reviewers:
Please check that the links on the landing page lead to pages that make sense for you.