Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

N1C certificates UI. Include screenshot.#259

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
mjang wants to merge3 commits intomain
base:main
Choose a base branch
Loading
fromfix-add-screenshot-certs

Conversation

mjang
Copy link
Contributor

@mjangmjang commentedMar 5, 2025
edited
Loading

Proposed changes

Include a screenshot to clarify the options associated with Certificates in NGINX One Console. Discussed internally withal.dacosta@f5.com

Review in this build:https://frontdoor-test-docs.nginx.com/previews/docs/259/nginx-one/how-to/nginx-configs/add-file/

Checklist

Before merging a pull request, run through this checklist and mark each as complete.

  • I have read thecontributing guidelines
  • I have signed theF5 Contributor License Agreement (CLA)
  • I have rebased my branch onto main
  • I have ensured my PR is targeting the main branch and pulling from my branch from my own fork
  • I have ensured that the commit messages adhere toConventional Commits
  • I have ensured that documentation content adheres tothe style guide
  • If the change involves potentially sensitive changes1, I have assessed the possible impact
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • I have ensured that existing tests pass after adding my changes
  • If applicable, I have updatedREADME.md andCHANGELOG.md

Footnotes

  1. Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer toour style guide for guidance about placeholder content.

@mjangmjang self-assigned thisMar 5, 2025
@mjangmjang added documentationImprovements or additions to documentation customer-feedback product/nginx-oneNGINX One Console labelsMar 5, 2025
@github-actionsGitHub Actions
Copy link

Deploy Preview will be available once build job completes!

NameLink
😎 Deploy Previewhttps://frontdoor-test-docs.nginx.com/previews/docs/259/

Co-authored-by: Alan Dooley <a.dooley@f5.com>
@mjangmjang marked this pull request as ready for reviewMarch 5, 2025 17:37
@mjangmjang requested a review froma team as acode ownerMarch 5, 2025 17:37
@@ -44,6 +44,8 @@ You can use the NGINX One Console to add a file to a specific instance. To do so

You now have multiple options, described in the sections which follow.

{{< img src="nginx-one/images/add-file-to-config.png" alt="A modal window to edit configuration, with a 'New Configuration File' option selected." >}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Can this be a bulleted list of the options instead of a screenshot?

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ADubhlaoichADubhlaoichADubhlaoich left review comments

@travisamartintravisamartintravisamartin left review comments

@y82y82y82 approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

@mjangmjang

Labels
customer-feedbackdocumentationImprovements or additions to documentationproduct/nginx-oneNGINX One Console
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@mjang@ADubhlaoich@y82@travisamartin

[8]ページ先頭

©2009-2025 Movatter.jp