Skip to content

fix: navbar active links, openapi error handling, memory leaks, and t… - #266

Open
AliceTenni wants to merge 1 commit into
Pidoko257:mainfrom
AliceTenni:fix/navbar-openapi-memory-leaks-and-testing-guide
Open

fix: navbar active links, openapi error handling, memory leaks, and t…#266
AliceTenni wants to merge 1 commit into
Pidoko257:mainfrom
AliceTenni:fix/navbar-openapi-memory-leaks-and-testing-guide

Conversation

@AliceTenni

Copy link
Copy Markdown

…esting guide

Fixes #205, #201, #207, and #198

Changes:

  • Add navbar active link highlighting with CSS styling (Fix Navbar Links Not Highlighting Active Page #205)

    • Active links now display with green color, font-weight 600, and bottom border
    • Docusaurus automatically applies navbar__link--active class
  • Handle missing openapi.yaml gracefully (Fix OpenAPI Spec Loading Errors When File is Missing #201)

    • Check for spec file existence before rendering
    • Show helpful error message with setup instructions instead of crashing
    • Prevents portal crash when spec is missing
  • Fix memory leaks in API testing widget (Fix Memory Leaks in API Testing Widget #207)

    • Add AbortController to cancel fetch requests on unmount
    • Implement DOM cleanup to remove Redoc event listeners
    • Add container ref for proper lifecycle management
    • Reduce memory overhead with optimized Redoc config options
  • Create comprehensive integration testing guide (Create Integration Testing Setup Guide #198)

    • Setup instructions with Jest configuration
    • Example test patterns: validation, errors, rate limiting, transactions
    • Best practices: test isolation, fixtures, helper functions, naming
    • Environment configuration and CI/CD examples
    • Troubleshooting section

Closes #205
Closes #198
Closes #207
Closes #201

…esting guide

Fixes Pidoko257#205, Pidoko257#201, Pidoko257#207, and Pidoko257#198

Changes:
- Add navbar active link highlighting with CSS styling (Pidoko257#205)
  * Active links now display with green color, font-weight 600, and bottom border
  * Docusaurus automatically applies navbar__link--active class

- Handle missing openapi.yaml gracefully (Pidoko257#201)
  * Check for spec file existence before rendering
  * Show helpful error message with setup instructions instead of crashing
  * Prevents portal crash when spec is missing

- Fix memory leaks in API testing widget (Pidoko257#207)
  * Add AbortController to cancel fetch requests on unmount
  * Implement DOM cleanup to remove Redoc event listeners
  * Add container ref for proper lifecycle management
  * Reduce memory overhead with optimized Redoc config options

- Create comprehensive integration testing guide (Pidoko257#198)
  * Setup instructions with Jest configuration
  * Example test patterns: validation, errors, rate limiting, transactions
  * Best practices: test isolation, fixtures, helper functions, naming
  * Environment configuration and CI/CD examples
  * Troubleshooting section
@drips-wave

drips-wave Bot commented Jul 27, 2026

Copy link
Copy Markdown

@AliceTenni Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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

Labels

None yet

Projects

None yet

1 participant