Skip to content

Conversation

@micjohnson777
Copy link
Contributor

This PR updates the content in the ACME DNS Authenticator and CSR tutorials to show correct information on adding the ACME DNS authenticator, adding a CSR, and creating an ACME certificate for the CSR.
It also updates the CredentialsCertificatesScreen.png to show each widget with added items.

Backports to 25.10

Thanks for contributing to TrueNAS documentation! By opening a Pull Request, you're acknowledging that your changes will be distributed under the Creative Commons 4.0 license.

This commit updates the instructions in the AddACMESCALE.md tutorial.
This commit updates the tutorial for adding a CSR and ACME Certificate.
This commit changes the image to show each widget populated (certificate, csr, and acme dns authenticator)
@micjohnson777 micjohnson777 requested a review from a team as a code owner October 16, 2025 18:21
@bugclerk bugclerk changed the title Pd 2255 update acme certificate content PD-2255 / 26.04 / Pd 2255 update acme certificate content Oct 16, 2025
@bugclerk
Copy link
Contributor

@github-actions
Copy link

github-actions bot commented Oct 16, 2025

Vale Style Guide Issues (Previous - Click to expand)

Vale Style Guide Issues

📊 Summary: 45 issues found in changed files - 3 errors, 0 warnings, 42 suggestions

Changed files with issues:

content/SCALE/SCALETutorials/Credentials/Certificates/AddCSRsSCALE.md (27 issues)

  • Lines 16, 21, 25, 97 - suggestion: Keep sentences under 30 words
  • Lines 21, 23, 29, 59, 90, 93, 94, 109 - suggestion: Spell out 'ACME' on first use
  • Lines 23, 94, 103 - suggestion: Use active voice instead of passive voice
  • Lines 25, 97 - error: Don't add '(s)' to a singular noun. Use plural instead.
  • Line 88 - suggestion: Spell out 'ACNE' on first use
  • Line 109 - suggestion: Spell out 'URI' on first use

content/SCALE/SCALETutorials/Credentials/Certificates/AddACMESCALE.md (18 issues)

  • Lines 2, 16, 20, 23, 28, 33, 55, 57, 69 - suggestion: Spell out 'ACME' on first use
  • Line 20 - suggestion: Spell out 'AWS' on first use
  • Line 27 - suggestion: Use 'login' instead of 'log in'
  • Line 27 - error: Use 'login' instead of 'log in'
  • Line 51 - suggestion: Spell out 'OVH' on first use
  • Line 63 - suggestion: Use active voice instead of passive voice

🔧 To fix: Review the issues above and update your documentation accordingly.

@linzi-ix linzi-ix added the Technical Writer Review (LM) A technical writer is reviewing the PR. label Oct 22, 2025
@github-actions
Copy link

github-actions bot commented Oct 22, 2025

Vale Style Guide Issues (Previous - Click to expand)

Vale Style Guide Issues

📊 Summary: 45 issues found in changed files - 3 errors, 0 warnings, 42 suggestions

Changed files with issues:

content/SCALE/SCALETutorials/Credentials/Certificates/AddCSRsSCALE.md (27 issues)

  • Lines 16, 21, 25, 97 - suggestion: Keep sentences under 30 words
  • Lines 21, 23, 29, 59, 90, 93, 94, 109 - suggestion: Spell out 'ACME' on first use
  • Lines 23, 94, 103 - suggestion: Use active voice instead of passive voice
  • Lines 25, 97 - error: Don't add '(s)' to a singular noun. Use plural instead.
  • Line 88 - suggestion: Spell out 'ACNE' on first use
  • Line 109 - suggestion: Spell out 'URI' on first use

content/SCALE/SCALETutorials/Credentials/Certificates/AddACMESCALE.md (18 issues)

  • Lines 2, 16, 20, 23, 28, 33, 55, 57, 69 - suggestion: Spell out 'ACME' on first use
  • Line 20 - suggestion: Spell out 'AWS' on first use
  • Line 27 - suggestion: Use 'login' instead of 'log in'
  • Line 27 - error: Use 'login' instead of 'log in'
  • Line 51 - suggestion: Spell out 'OVH' on first use
  • Line 63 - suggestion: Use active voice instead of passive voice

🔧 To fix: Review the issues above and update your documentation accordingly.

@github-actions
Copy link

Vale Style Guide Issues

📊 Summary: 45 issues found in changed files - 3 errors, 0 warnings, 42 suggestions

Changed files with issues:

content/SCALE/SCALETutorials/Credentials/Certificates/AddCSRsSCALE.md (27 issues)

  • Lines 16, 21, 25, 97 - suggestion: Keep sentences under 30 words
  • Lines 21, 23, 29, 59, 90, 93, 94, 109 - suggestion: Spell out 'ACME' on first use
  • Lines 23, 94, 103 - suggestion: Use active voice instead of passive voice
  • Lines 25, 97 - error: Don't add '(s)' to a singular noun. Use plural instead.
  • Line 88 - suggestion: Spell out 'ACNE' on first use
  • Line 109 - suggestion: Spell out 'URI' on first use

content/SCALE/SCALETutorials/Credentials/Certificates/AddACMESCALE.md (18 issues)

  • Lines 2, 16, 20, 23, 28, 33, 55, 57, 69 - suggestion: Spell out 'ACME' on first use
  • Line 20 - suggestion: Spell out 'AWS' on first use
  • Line 27 - error: Use 'login' instead of 'log in'
  • Line 27 - suggestion: Use 'login' instead of 'log in'
  • Line 51 - suggestion: Spell out 'OVH' on first use
  • Line 63 - suggestion: Use active voice instead of passive voice

🔧 To fix: Review the issues above and update your documentation accordingly.

@linzi-ix linzi-ix merged commit b5ed0c2 into master Oct 22, 2025
5 checks passed
@linzi-ix linzi-ix deleted the PD-2255-Update-ACME-Certificate-Content branch October 22, 2025 15:05
@bugclerk
Copy link
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@truenas truenas locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants