Skip to content

Analyze NGINX Plus Deployment Guide: Global Server Load Balancing #96

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

Open
mjang opened this issue Jan 23, 2025 · 3 comments
Open

Analyze NGINX Plus Deployment Guide: Global Server Load Balancing #96

mjang opened this issue Jan 23, 2025 · 3 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@mjang
Copy link
Contributor

mjang commented Jan 23, 2025

This page, https://docs.nginx.com/nginx/deployment-guides/amazon-web-services/route-53-global-server-load-balancing/ has not been revised since April 2018.

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.

Alternatively, 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 23, 2025
@alai97
Copy link
Contributor

alai97 commented Mar 6, 2025

Hi @mjang, I'd be happy to work on this issue.

@mjang
Copy link
Contributor Author

mjang commented Mar 11, 2025

Thanks so much, @alai97 ! We appreciate your contributions so far :) For the record, I've assigned this issue to you.

@mjang
Copy link
Contributor Author

mjang commented Mar 11, 2025

@alai97 Thank you so much for "making us look" at this page! One thing I need to tell you: with this PR, we've chosen to "unpublish" this page, due to 18 missing screenshots

This page is still available in our repository. You're welcome to continue your work on this page. Hint: we believe that we can reduce (or maybe even eliminate) screenshots. If you continue to work on this issue -- if you can reduce the number of required screenshots, I think that'd make a great impression on potential future employers.

Alternatively, you could work on #100, which depicts a similar process. FYI, you can find the source for that page here: https://github.com/nginx/documentation/blob/main/content/nginx/deployment-guides/global-server-load-balancing/ns1-global-server-load-balancing.md

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

No branches or pull requests

2 participants