Skip to content

Example React first Initial Update#267

Open
sadeeqrabiu wants to merge 2 commits intofedimint:mainfrom
sadeeqrabiu:proxy
Open

Example React first Initial Update#267
sadeeqrabiu wants to merge 2 commits intofedimint:mainfrom
sadeeqrabiu:proxy

Conversation

@sadeeqrabiu
Copy link
Copy Markdown

##Overview
Reference to issue #250
This PR resolves the issue by refactoring the React Native example app from a single-screen layout into a clean, multi-screen architecture. This structure provides a much more realistic reference implementation of a wallet for developers integrating the SDK.

Changes Implemented:

  • Added React Navigation: Integrated @react-navigation/native and built out a structured navigation system.
  • Separated UI Logic into Logical Screens:
    • Wallet Overview Screen: Displays the current balance and high-level wallet information.
    • Send Screen: Dedicated interface for initiating transactions.
    • Receive Screen: Dedicated interface for generating invoices/receive requests.
    • Settings/History Screen: Separated additional wallet functionality for better organization.
  • Improved Code Structure: Cleaned up the main entry point to simply handle navigation routing, making the codebase more modular, readable, and easier to scale.

The example app now mimics the structure of a real-world wallet application, creating a much better developer experience for anyone using the SDK as a reference.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 5, 2026

⚠️ No Changeset found

Latest commit: 92bd082

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown
Member

@MrImmortal09 MrImmortal09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The design seems a bit off, and also it would be better if we could have a transaction history as well.

Also the wallet status is not being reflected correctly.

import React from 'react'
import { SafeAreaView, StatusBar, Text } from 'react-native'
import { NavigationContainer, DefaultTheme } from '@react-navigation/native'
import { createBottomTabNavigator } from '@react-navigation/bottom-tabs'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guess you have not pushed the updated package.json

@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​react-navigation/​bottom-tabs@​6.6.19910075100100
Addednpm/​@​react-navigation/​native@​6.1.18991007599100
Addednpm/​react@​19.2.41001008497100

View full report

@sadeeqrabiu
Copy link
Copy Markdown
Author

@MrImmortal09 I Refined the UI with a bright claymorphism style and improved overall design. Connected the history screen to transaction data for accurate display. Enhanced wallet sync logic for better state handling, updated navigation dependencies.

@MrImmortal09
Copy link
Copy Markdown
Member

@sadeeqrabiu There is still some dependency issue, and Could you please add an onboarding screen as well?

You can refer the expo-app example as well.

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.

2 participants