Skip to content

Conversation

lamATnginx
Copy link
Contributor

@lamATnginx lamATnginx commented Sep 10, 2025

Proposed changes

  • Replaced broken jQuery usage with vanilla JS.

Before:
Screenshot 2025-09-10 at 1 55 08 PM

After:
Screenshot 2025-09-10 at 1 53 21 PM

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.

@lamATnginx lamATnginx self-assigned this Sep 10, 2025
@lamATnginx lamATnginx added the bug Something isn't working label Sep 10, 2025
@lamATnginx lamATnginx requested a review from a team as a code owner September 10, 2025 20:47
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 10, 2025
Copy link

Deploy Preview will be available once build job completes!

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

@ryepup
Copy link
Contributor

ryepup commented Sep 10, 2025

code LGTM, but I can't test it in the preview; looks like we're loading the script file via <script type="module" src="/nginxaas-azure/js/cost-calculator_v2.js"></script> so it's not picking up the preview path

Copy link
Contributor

@travisamartin travisamartin left a comment

Choose a reason for hiding this comment

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

Verified locally -- LGTM.

@lamATnginx lamATnginx merged commit 9fc4b77 into main Sep 10, 2025
15 checks passed
@lamATnginx lamATnginx deleted the fix-n4a-calc branch September 10, 2025 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants