Skip to content

Conversation

@xynicole
Copy link
Contributor

@xynicole xynicole commented Dec 30, 2025

Proposed changes

update NGINXaaS Cost Estimator with Tiered pricing
image

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@xynicole xynicole requested a review from a team as a code owner December 30, 2025 10:24
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Dec 30, 2025
@xynicole xynicole requested a review from JTorreG December 30, 2025 10:26
@github-actions
Copy link

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/1625/

@xynicole xynicole force-pushed the 904-estimator branch 2 times, most recently from d17bd35 to 3eff966 Compare January 2, 2026 23:27
@nginx-jack
Copy link
Contributor

Can you run the playwright tests for this, and make sure they're passing?
cd tests && npx playwright test
@lamATnginx can you help our here if there are any issues?

Copy link
Contributor

@lamATnginx lamATnginx left a comment

Choose a reason for hiding this comment

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

Code changes look good for the most part and thank you for adding them, but some test need to be added:

  1. Test added similar to https://github.com/nginx/documentation/pull/1596/files#diff-44961457451d64b0d5be0f5d6964247d67eb8188ea0754e574e8b26d68e12ce7
  2. Update the github action's white-listed paths to include the paths to source code for Google calculator: https://github.com/nginx/documentation/pull/1596/files#diff-7afcd2d8f7b49bda74843f209eefb7b2da45f7e7803bf2e4bd636699b76aa2d3R6

@github-actions github-actions bot added the tooling Back end, repository, Hugo, and all things not related to content label Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation tooling Back end, repository, Hugo, and all things not related to content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants