feat: wire frontend to real backend API#162
Open
fawizzy wants to merge 6 commits intoStayLitCodes:mainfrom
Open
feat: wire frontend to real backend API#162fawizzy wants to merge 6 commits intoStayLitCodes:mainfrom
fawizzy wants to merge 6 commits intoStayLitCodes:mainfrom
Conversation
- Add lib/api-client.ts with NEXT_PUBLIC_API_URL config, JWT injection, and automatic token refresh on 401 - Add services/auth.ts implementing the challenge → sign → verify auth flow - Replace all mock data in EscrowService with real HTTP calls (GET/POST /escrows and all sub-routes) - Replace mock useWallet hook with real useWalletConnection from WalletContext - Convert useEscrow from useState/useEffect to useQuery with 404-aware retry logic - Add signMessage to FreighterService, IWalletService, and WalletContext for auth signing - Mount WalletProvider in Providers tree - Add .env.example documenting NEXT_PUBLIC_API_URL and NEXT_PUBLIC_STELLAR_NETWORK
|
@fawizzy 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! 🚀 |
Contributor
|
Fix failing workflow |
- Rewrite eslint.config.js as a valid ESLint 10 flat config (remove mixed ESM/CJS, replace invalid env/extends keys with plugins/rules) - Add "type": "module" to root package.json so Node loads the ESM config - Remove --ext flag from root lint script (dropped in ESLint 9+) - Add "root": true to frontend .eslintrc.json to prevent ESLint from traversing up to the root config - Downgrade no-explicit-any and no-unused-vars to warn so pre-existing violations don't block CI; suppress stale eslint-disable directives
Author
|
fixed the failing workflow |
Contributor
|
Kindly fix workflow |
- Remove unused imports and variables throughout - Replace `any` types with proper types (unknown, Record<string, unknown>, typed interfaces) - Fix unused catch bindings with empty catch clauses - Type Stellar server, wallet interfaces, and error handlers properly - Configure frontend ESLint to honour underscore-prefixed ignore pattern
- Resolve conflict in escrow.service.ts (actorId optional, Record<string, any>) - Accept upstream deletion of eslint.config.js
Contributor
|
Please fix conflict |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
services/escrow.tswith real HTTP calls to theNestJS backend, covering every escrow endpoint (list, detail, create, fund,
release, cancel, dispute, fulfill/confirm conditions, events)
lib/api-client.ts— a typed fetch wrapper with configurableNEXT_PUBLIC_API_URL, automaticAuthorization: Bearerheader injection, andsilent token refresh on 401
services/auth.tsimplementing the full wallet auth flow:POST /auth/challenge→ sign message with wallet →POST /auth/verify→ store JWTsignMessage()toFreighterService,IWalletService, andWalletContextto support signing the raw auth challenge (distinct from XDRtransaction signing)
hooks/useWallet.ts(hardcoded public key) with a realwrapper over
useWalletConnectionfromWalletContexthooks/useEscrow.tsfrom manualuseState/useEffect/fetchtouseQuerywith 404-aware retry logicWalletProviderincomponent/Providers.tsxso wallet state isavailable app-wide
apps/frontend/.env.exampledocumenting required env varscloses #141