Skip to content
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

Update K8s docs homepage #13230

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Update K8s docs homepage #13230

merged 1 commit into from
Oct 12, 2023

Conversation

evilnick
Copy link
Contributor

Done

  • Reworked the landing page to standardise on diataxis layout.

QA

  • Check out this feature branch
  • Run the site using the command ./run serve or dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8001/
    • Be sure to test on mobile, tablet and desktop screen sizes

@webteam-app
Copy link

Demo starting at https://ubuntu-com-13230.demos.haus

@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #13230 (67f31fd) into main (2cc24d9) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #13230   +/-   ##
=======================================
  Coverage   75.31%   75.31%           
=======================================
  Files         106      106           
  Lines        2840     2840           
  Branches      929      929           
=======================================
  Hits         2139     2139           
  Misses        679      679           
  Partials       22       22           

Copy link
Contributor

@carkod carkod left a comment

Choose a reason for hiding this comment

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

LGTM

@carkod carkod merged commit 5f8930e into canonical:main Oct 12, 2023
@evilnick evilnick deleted the k8s-homepage branch October 12, 2023 19:25
@anthonydillon
Copy link
Contributor

@evilnick none of these links work or seem to have destinations to go to.

@carkod did you check the links?

Can we revert this change or update the links please?

@evilnick
Copy link
Contributor Author

sorry, this all got a bit messy - the pages they link to are in #13254

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants