Skip to content
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

Refactor API URLs to use environment variables #85

Merged

Conversation

sakshamgupta912
Copy link

Title: Environment Configuration for Dynamic Base URL Setup

Description

This PR adds an environment-based configuration to dynamically manage the base API URL. Previously, the URLs were hardcoded (http://localhost:5000). With this update, the serverUrl from the environment file is used, allowing for different base URLs based on the environment (e.g., development, production).

Related Issue

Issue #84

Motivation and Context

This change is required to make the codebase more flexible and environment-agnostic. By using the environment configuration, switching between environments (development, testing, production) is streamlined without modifying individual components. This also aligns with best practices for scalable applications.

Screenshots (In case of UI changes)

N/A

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • ✅ New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • ✅ My code follows the code style of this project.
  • ✅ My change requires a change to the documentation.
  • ✅ I have updated the documentation accordingly.
  • ✅ All new and existing tests have passed and it does not give any unexpected error for the same.

@sakshamgupta912
Copy link
Author

@rajutkarsh07 rajutkarsh07 merged commit 27a5d25 into c2siorg:feature/webiu-2024 Oct 29, 2024
1 of 2 checks passed
Rajshah1302 pushed a commit to Rajshah1302/Webiu_Raj that referenced this pull request Dec 15, 2024
Refactor API URLs to use environment variables
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants