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 copy on kubernetes/install #13581

Merged
merged 2 commits into from
Feb 15, 2024
Merged

Update copy on kubernetes/install #13581

merged 2 commits into from
Feb 15, 2024

Conversation

petesfrench
Copy link
Contributor

Done

  • Updates /kubernetes/install based on the copydoc

QA

  • Go to
  • Compare against the copydoc and resolve comments as you go

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-8850

Screenshots

[If relevant, please include a screenshot.]

Help

QA steps - Commit guidelines

@webteam-app
Copy link

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

Copy link

codecov bot commented Feb 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (95aa5d6) 74.41% compared to head (baec94e) 74.41%.
Report is 13 commits behind head on k8s-bubble-refresh.

Additional details and impacted files
@@                 Coverage Diff                 @@
##           k8s-bubble-refresh   #13581   +/-   ##
===================================================
  Coverage               74.41%   74.41%           
===================================================
  Files                     107      107           
  Lines                    2838     2838           
  Branches                  946      946           
===================================================
  Hits                     2112     2112           
  Misses                    702      702           
  Partials                   24       24           

@petesfrench petesfrench changed the base branch from main to k8s-bubble-refresh February 14, 2024 11:50
Copy link
Contributor

@akbarkz akbarkz left a comment

Choose a reason for hiding this comment

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

You also missed one more change from the copy doc, which is removing version number from "Multi-node, highly available Kubernetes with MicroK8s" section

templates/kubernetes/install.html Show resolved Hide resolved
templates/kubernetes/install.html Outdated Show resolved Hide resolved
templates/kubernetes/install.html Outdated Show resolved Hide resolved
templates/kubernetes/install.html Outdated Show resolved Hide resolved
templates/kubernetes/install.html Outdated Show resolved Hide resolved
@@ -477,14 +523,14 @@ <h2>Kubernetes resources</h2>
<p class="p-heading-icon__title p-muted-heading" style="color: #fff;">Data Sheet</p>
</div>
</div>
<p><a class="p-link--inverted" href="https://assets.ubuntu.com/v1/b5f9ae49-Enterprise_Kubernetes_Datasheet.pdf">Kubernetes for the enterprise</a></p>
<p><a class="p-link--inverted" href="https://assets.ubuntu.com/v1/b5f9ae49-Enterprise_Kubernetes_Datasheet.pdf">Kubernetes for the enterprise&nbsp;&rsaquo;</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

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

This change is not specified in the copy doc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Chevron usage is not decided on the copydoc level, but on the design level. usually there is some consistency of use within a page, so In this case I was trying to get all the links to align with one another

@@ -496,7 +542,7 @@ <h2>Kubernetes resources</h2>
<p class="p-heading-icon__title p-muted-heading" style="color: #fff;">Industry Report</p>
</div>
</div>
<p><a class="p-link--inverted" href="https://juju.is/cloud-native-kubernetes-usage-report-2021">The Kubernetes and cloud native operations report 2021</a></p>
<p><a class="p-link--inverted" href="https://juju.is/cloud-native-kubernetes-usage-report-2022">The Kubernetes and cloud native operations report 2022&nbsp;&rsaquo;</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<p><a class="p-link--inverted" href="https://juju.is/cloud-native-kubernetes-usage-report-2022">The Kubernetes and cloud native operations report 2022&nbsp;&rsaquo;</a></p>
<p><a class="p-link--inverted" href="https://juju.is/cloud-native-kubernetes-usage-report-2022">The Kubernetes and cloud native operations report 2022</a></p>

@@ -515,7 +561,7 @@ <h2>Kubernetes resources</h2>
<p class="p-heading-icon__title p-muted-heading" style="color: #fff;">Whitepaper</p>
</div>
</div>
<p><a class="p-link--inverted" href="/engage/kubernetes-deployment-enterprise-whitepaper">Five strategies to accelerate Kubernetes deployment in the enterprise</a></p>
<p><a class="p-link--inverted" href="/engage/kubernetes-deployment-enterprise-whitepaper">Five strategies to accelerate Kubernetes deployment in the enterprise&nbsp;&rsaquo;</a></p>
Copy link
Contributor

Choose a reason for hiding this comment

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

This change is not specified in the copy doc

@petesfrench petesfrench merged commit 50c7344 into k8s-bubble-refresh Feb 15, 2024
26 of 27 checks passed
@petesfrench petesfrench deleted the wd-8850 branch February 15, 2024 13:12
akbarkz pushed a commit that referenced this pull request Feb 19, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Feb 21, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Feb 23, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Feb 26, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Feb 27, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Mar 4, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Mar 12, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Mar 20, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Apr 1, 2024
* Update copy on kubernetes/install

* Update with review suggestions
akbarkz pushed a commit that referenced this pull request Apr 11, 2024
* Update copy on kubernetes/install

* Update with review suggestions
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.

None yet

3 participants