Skip to content

config (api path) fixes#13

Merged
HardMax71 merged 1 commit intomainfrom
dev
Dec 8, 2025
Merged

config (api path) fixes#13
HardMax71 merged 1 commit intomainfrom
dev

Conversation

@HardMax71
Copy link
Owner

@HardMax71 HardMax71 commented Dec 8, 2025

Summary by CodeRabbit

  • Chores
    • Reorganized API endpoint configuration and paths for improved consistency across the application.

✏️ Tip: You can customize this high-level summary in your review settings.

@HardMax71 HardMax71 merged commit f194597 into main Dec 8, 2025
3 of 4 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Dec 8, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The frontend API endpoints are restructured by moving the /api path segment from individual endpoint URLs to the base API URL. The base URL changes from http://localhost:8000 to http://localhost:8000/api, allowing all downstream API calls to use shorter paths (e.g., /v1/health/ instead of /api/v1/health/).

Changes

Cohort / File(s) Summary
Environment & Client Configuration
.env.example, frontend/src/api/client.ts
Updated VITE_API_BASE_URL and API_BASE_URL from http://localhost:8000 to http://localhost:8000/api
Frontend Pages
frontend/src/pages/Health.tsx, frontend/src/pages/JobStatus.tsx, frontend/src/pages/Landing.tsx, frontend/src/pages/PrivacyPolicy.tsx
Updated API endpoint paths to remove /api prefix (e.g., /api/v1/health//v1/health/, /api/docs//docs/)
Services
frontend/src/services/resumeService.ts
Updated uploadResume endpoint from ${API_BASE_URL}/api/v1/resumes/ to ${API_BASE_URL}/v1/resumes/

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • All changes follow a consistent pattern: moving /api from endpoint strings to the base URL
  • URL string updates are straightforward with no logic modifications
  • Changes are homogeneous despite being spread across 6 files, reducing cognitive load

Possibly related PRs

  • Added publish scripts  #6: Updates environment configuration for the frontend API base URL, sharing the same goal of adjusting VITE_API_BASE_URL defaults across the project.

Poem

🐰 The paths grew shorter, cleaner still,
As /api climbed the base URL hill,
No more redundant prefixes to chase,
Each endpoint found its rightful place!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 18c8cbe and 593d4be.

📒 Files selected for processing (7)
  • .env.example (1 hunks)
  • frontend/src/api/client.ts (1 hunks)
  • frontend/src/pages/Health.tsx (1 hunks)
  • frontend/src/pages/JobStatus.tsx (1 hunks)
  • frontend/src/pages/Landing.tsx (3 hunks)
  • frontend/src/pages/PrivacyPolicy.tsx (1 hunks)
  • frontend/src/services/resumeService.ts (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant