Refactor third-party integrations to use a shared HTTP client#171
Conversation
|
@vraj826 is attempting to deploy a commit to the Dot_NotSam's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@SamXop123 Can you review the changes! |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
SamXop123
left a comment
There was a problem hiding this comment.
Looks solid! thank you for the contribution
|
🎉 Congratulations @vraj826! 🎉 Your pull request has been successfully merged into samdev-pulse! Thank you so much for your valuable contribution and effort. Every single improvement helps make samdev-pulse a better profile experience for everyone! 🙌 🚀 What's Next?
💬 Stay Connected: Thank you again, and keep up the amazing work! 💻✨ |
Description
Introduced a shared HTTP client and refactored all third-party integrations to use a common request layer. This centralizes timeout handling, response validation, JSON parsing, and error normalization while preserving existing fallback behavior.
Related Issue
Closes #163
Type of Change
Changes Made
src/utils/http-client.js) for external API communicationTesting
npm test— all tests passVerification performed:
npm.cmd run lint→ passingnpm.cmd run test:unit→ passing (12 suites, 141 tests)Screenshots (if applicable)
N/A
Checklist