Skip to content

Conversation

@technologic-technologic
Copy link
Owner

Updated the default port in which te client runs to the specified in the requirements port 8080.

Key changes

  • Updated server to 8080 in vite.config.ts file.
  • Upgraded proxy specifications to cleaner requests and behavior.

No breaking changes, yet.

Updated the default port in which te client runs to the specified in the requirements port 8080.

## Key changes
- Updated server to 8080 in vite.config.ts file.
- Upgraded proxy specifications to cleaner requests and behavior.

No breaking changes, yet.
@github-actions
Copy link

PR Summary

WHAT Changed

  • Updated API endpoint URLs in Requests.ts to use a relative path (/api/...) instead of a hardcoded base URL (http://localhost:9090).
  • Modified vite.config.ts to include a development server configuration with a proxy for API requests.

WHY It Matters

  • The change to relative paths allows for easier deployment and testing in different environments without needing to modify the base URL.
  • The proxy configuration in vite.config.ts facilitates local development by redirecting API calls to the backend server, improving the development workflow.

RISKS

  • Potential issues with API calls if the backend server is not running or accessible at the specified target.
  • Changes to the endpoint structure may lead to broken functionality if any components rely on the old URL format.

TESTS to Add

  • Ensure that all API calls return the expected responses when using the new relative paths.
  • Test the proxy configuration to verify that API requests are correctly routed to the backend server during development.

BREAKING CHANGES

  • The change in API endpoint structure may require updates in any components or services that directly reference the old base URL.

Changed File Paths

  • inventory-manager/src/services/Requests.ts
  • inventory-manager/vite.config.ts

@technologic-technologic technologic-technologic added bug Something isn't working enhancement New feature or request labels Oct 12, 2025
@technologic-technologic technologic-technologic added this to the Requirements milestone Oct 12, 2025
@technologic-technologic technologic-technologic linked an issue Oct 12, 2025 that may be closed by this pull request
@technologic-technologic technologic-technologic merged commit 715e862 into 26-migrate-app-from-cra-to-vite Oct 12, 2025
1 check passed
@technologic-technologic technologic-technologic deleted the 23-change-port-to-8080 branch October 12, 2025 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change client default port to 8080

2 participants