Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create profile screen #17

Open
wants to merge 29 commits into
base: main
Choose a base branch
from
Open

Create profile screen #17

wants to merge 29 commits into from

Conversation

jessepinho
Copy link
Collaborator

@jessepinho jessepinho commented Jan 2, 2025

Screen.Recording.2025-01-03.at.3.44.34.PM.mov

At a high level, this PR creates the initial profile screen, and the gRPC/default payment token settings screens. But it also includes a number of new technologies under the hood that make those screens possible.

In this PR

  • Install and configure redux-persist/redux-persist-expo-securestore, so as to persist the secureStore Redux slice to the device's encrypted storage.
  • Write README docs on how to use secure storage.
  • Create various utility components, like <Box />, <TextInputIconStartAdornment />, <ListItemIconSuffix />, etc.
  • Create the profile screen and the "Default payment token"/"RPC" sub-screens.
  • Link to the profile screen from the user avatar at top left.
  • Create a bunch more props for <TextInput />, and create a Storybook story.

@jessepinho jessepinho force-pushed the jessepinho/profile-screen branch 2 times, most recently from b1b80fe to 0d56d26 Compare January 3, 2025 01:23
"bundleIdentifier": "com.margulus.reactnativeexpo"
"bundleIdentifier": "com.margulus.reactnativeexpo",
"config": {
"usesNonExemptEncryption": false
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

See https://docs.expo.dev/versions/latest/sdk/securestore/#exempting-encryption-prompt

(I can't leave this comment in the file, since JSON doesn't support comments 😭 )

@jessepinho jessepinho marked this pull request as ready for review January 3, 2025 23:45
@jessepinho jessepinho requested a review from TalDerei January 3, 2025 23:45
Base automatically changed from jessepinho/portfolio-screen to jessepinho/more-screens January 6, 2025 20:30
Base automatically changed from jessepinho/more-screens to main January 6, 2025 20:30
@jessepinho jessepinho force-pushed the jessepinho/profile-screen branch from 6f47cd1 to b53d703 Compare January 6, 2025 20:32
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.

1 participant