Skip to content

Analyze NGINX Plus Deployment Guide: Active-Active HA #75

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

Closed
mjang opened this issue Jan 16, 2025 · 3 comments · Fixed by #176
Closed

Analyze NGINX Plus Deployment Guide: Active-Active HA #75

mjang opened this issue Jan 16, 2025 · 3 comments · Fixed by #176
Assignees
Labels
good first issue Good for newcomers

Comments

@mjang
Copy link
Contributor

mjang commented Jan 16, 2025

This page, https://docs.nginx.com/nginx/deployment-guides/amazon-web-services/high-availability-network-load-balancer/ has not been revised since March 2020.

You can review and edit the source code for the page directly in our repository. You can edit it directly, as described in Editing files. If you know Git, you're welcome to fork the repository and use your skills to set up a Pull request.

We need to audit this page.

Here's what we'd like you to do (tasks):

  • Audit screenshots

    • Update if they're out of date
    • Evaluate the screenshots:
      • Do they include too much info, such as Personally Identifying Information
      • Do we need all the information in the screenshot
      • Do we even need the screenshot, since it's from "someone else's" UI
  • Evaluate the page for readability

If you're not (yet) comfortable with Git, you're welcome to post your answers to this issue in a Discussion.

  • We'll be happy to give you credit!

If you haven't been able to do everything in this issue, that's OK. We appreciate all the help that we can get.

@mjang mjang added the good first issue Good for newcomers label Jan 16, 2025
@11t97
Copy link
Contributor

11t97 commented Feb 7, 2025

Hi @mjang, I can take the readability evaluation and changes for this one.

@mjang
Copy link
Contributor Author

mjang commented Feb 7, 2025

Hi @mjang, I can take the readability evaluation and changes for this one.

Thanks! I look forward to your contribution! You'll notice that I've set your account as the "Assignee" for this issue.

FYI, while I'm on PTO (Feb 8-17), @ADubhlaoich should be available to answer questions -- or review your work :). You're also welcome to post questions in our Discussions forum..

11t97 added a commit to 11t97/documentation that referenced this issue Feb 7, 2025
Corrections to align with style guide recommendations for:
* click/select
* load balancing
* configure/ set up
* (radio) button
* page directions
11t97 added a commit to 11t97/documentation that referenced this issue Feb 7, 2025
Clean up a couple of missed clicks.
@11t97
Copy link
Contributor

11t97 commented Feb 7, 2025

Hi @mjang , enjoy your PTO! Hi @ADubhlaoich, please let me know if you have any questions about this first PR by me, or if I should make additional changes. Thanks! --Dianne

ADubhlaoich pushed a commit that referenced this issue Feb 18, 2025
- Style guide conformance
- Information architecture consistency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants