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

chore: add example app to rn-base #1171

Merged
merged 6 commits into from
Nov 22, 2024
Merged

chore: add example app to rn-base #1171

merged 6 commits into from
Nov 22, 2024

Conversation

moldy530
Copy link
Collaborator

@moldy530 moldy530 commented Nov 22, 2024

  • fix: fix linting and spacing issues
  • feat: remove package hoisting from example project
  • fix: fix linting and spacing issues
  • feat: add signer authentication sample implementation in example app
  • feat: add tsx to handle server instantiation
  • feat: update peer-dependencies for signer package

PR-Codex overview

This PR focuses on updating the @account-kit/react-native-signer library by modifying exports, enhancing server capabilities, and refactoring client-side code for improved navigation and session management.

Detailed summary

  • Changed export from RNSignerClient to RNAlchemySigner in index.tsx.
  • Added an Express server in redirect-server/index.ts for handling redirects.
  • Updated MainActivity.kt to support react-native-screens.
  • Refactored session management in manager.ts to include error handling.
  • Updated package.json scripts to include a server start command.
  • Enhanced navigation in App.tsx using @react-navigation/native.
  • Created HomeScreen.tsx for user authentication and display.
  • Updated AndroidManifest.xml to include new intent filters.
  • Modified dependencies and peerDependencies in package.json for compatibility.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

vercel bot commented Nov 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
aa-sdk-site 🔄 Building (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 6:50pm
aa-sdk-ui-demo 🔄 Building (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 6:50pm

Copy link

graphite-app bot commented Nov 22, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “graphite-merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@moldy530
Copy link
Collaborator Author

gotta remerge this because the history got messed up

@moldy530 moldy530 merged commit 881242b into moldy/rn-base Nov 22, 2024
2 of 7 checks passed
@moldy530 moldy530 deleted the iyk/e2e-demo branch November 22, 2024 18:51
useEffect(() => {
// get the user if already logged in
signer.getAuthDetails().then(setUser);
}, []);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

⚠️ [eslint] <react-hooks/exhaustive-deps> reported by reviewdog 🐶
React Hook useEffect has a missing dependency: 'signer'. Either include it or remove the dependency array.

const subscription = Linking.addEventListener("url", handleIncomingURL);

return () => subscription.remove();
}, []);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

⚠️ [eslint] <react-hooks/exhaustive-deps> reported by reviewdog 🐶
React Hook useEffect has a missing dependency: 'handleIncomingURL'. Either include it or remove the dependency array.

moldy530 added a commit that referenced this pull request Nov 22, 2024
* fix: fix linting and spacing issues

* feat: remove package hoisting from example project

* fix: fix linting and spacing issues

* feat: add signer authentication sample implementation in example app

* feat: add tsx to handle server instantiation

* feat: update peer-dependencies for signer package

---------

Co-authored-by: Iyk Azorji <[email protected]>
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