Skip to content

Api client#147

Merged
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
KAMALDEEN333:Api-Client
Apr 23, 2026
Merged

Api client#147
RUKAYAT-CODER merged 2 commits into
rinafcode:mainfrom
KAMALDEEN333:Api-Client

Conversation

@KAMALDEEN333
Copy link
Copy Markdown
Contributor

Summary

Related Issue

Description

Overview
Multiple API client implementations exist in different locations (src/lib/api.ts, src/app/mobile/services/api.ts).

Background
No single source of truth for API calls.

Specifications

Consolidate to single API client
Add interceptors
Tasks:

Merge mobile API into main client
Add request/response interceptors
Implement retry logic
Impacted Files:

src/lib/api.ts
src/app/mobile/services/api.ts
Labels: frontend, api, priority-medium

Acceptance Criteria
Single unified API client

Closes #123

Type of change

  • Feature
  • Bug fix
  • Chore / Refactor
  • Docs

Screenshots / Recording (if UI)

Testing

  • npm run type-check
  • npm run lint
  • npm run test
  • npm run build

Quality gate checklist

  • CI checks pass (Frontend CI)
  • At least 1–2 approvals (per branch protection rules)
  • Branch is up-to-date with the base branch
  • All conversations resolved
  • PR description includes Closes #<issue-number>

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Apr 22, 2026

@KAMALDEEN333 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

@RUKAYAT-CODER RUKAYAT-CODER merged commit d97a7d9 into rinafcode:main Apr 23, 2026
3 of 6 checks passed
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.

Duplicate API Client Implementation

2 participants