Skip to content

feat: unify wallet transaction lifecycle (#55 #56 #66)#86

Open
barry01-hash wants to merge 1 commit into
StepFi-app:mainfrom
barry01-hash:unified-transaction
Open

feat: unify wallet transaction lifecycle (#55 #56 #66)#86
barry01-hash wants to merge 1 commit into
StepFi-app:mainfrom
barry01-hash:unified-transaction

Conversation

@barry01-hash

Copy link
Copy Markdown

Closes #83

Summary

Closes #[issue number]

Briefly describe what this PR does in 2-3 sentences.

This repo is for the React web app only

This app targets sponsors, vendors, and mentors.
It does NOT serve learners. Learner features
belong in StepFi-App.

Before submitting, confirm your changes belong here:

  • My changes are inside src/
  • I have NOT added Rust, Soroban, or
    contract code
  • I have NOT added React Native or
    Expo-specific code
  • I have NOT hardcoded hex color values
    (use Tailwind classes or constants/colors.ts)
  • All icons are from lucide-react only
  • No API calls made directly in page files
    (use services/ layer only)

Type of change

  • Bug fix
  • New page or component
  • Service layer addition
  • Styling or responsive fix
  • Accessibility improvement
  • Performance improvement

Testing

  • npm run build passes with zero errors
  • npm run lint passes with zero errors
  • Loading, error, and empty states handled
  • Page tested on mobile viewport (375px)
  • No console errors in browser

Context files reviewed

  • context/architecture-context.md
  • context/code-standards.md
  • context/progress-tracker.md updated

Mandatory before requesting review

Running these must all exit 0:
npm run lint
npm test
npm run build

If either fails, fix it before opening this PR.
PRs that reduce the test count will be rejected.
PRs with failing CI will be closed without review.

@barry01-hash
barry01-hash requested a review from EmeditWeb as a code owner July 18, 2026 10:41
@barry01-hash

Copy link
Copy Markdown
Author

Please merge

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.

core: build unified transaction service for build-sign-submit-poll

1 participant