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

docs(hooks): add initial react hooks docs #561

Merged
merged 2 commits into from
Apr 10, 2024

Conversation

moldy530
Copy link
Collaborator

@moldy530 moldy530 commented Apr 10, 2024

Pull Request Checklist


PR-Codex overview

This PR adds new React hooks and configurations related to user authentication, signer status, and account management. It also updates TypeScript configurations and documentation.

Detailed summary

  • Added new React hooks for user authentication and account management
  • Updated TypeScript configurations for module and module resolution
  • Updated documentation for the useUser hook in a Markdown file

The following files were skipped due to too many changes: packages/alchemy/src/react/index.ts, packages/alchemy/src/react/hooks/useAuthenticate.ts, site/react/useBundlerClient.md, site/react/useSigner.md, site/react/useAuthenticate.md, site/react/useSignerStatus.md, site/react/createConfig.md, site/react/overview.md, site/react/useAccount.md, site/react/useSmartAccountClient.md

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

@@ -1,14 +1,14 @@
{
"compilerOptions": {
"baseUrl": ".",
"module": "esnext",
"module": "nodenext",
Copy link
Contributor

Choose a reason for hiding this comment

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

any reason this was needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yea the snippets files weren't correctly resolving the /react and /config imports so I did this to match what we have in other tsconfigs

Copy link
Contributor

@avasisht23 avasisht23 left a comment

Choose a reason for hiding this comment

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

small suggestions, but lgtm

site/snippets/react/app.tsx Show resolved Hide resolved
site/snippets/react/useSmartAccountClient.tsx Outdated Show resolved Hide resolved
site/snippets/react/useBundlerClient.tsx Show resolved Hide resolved
site/react/useAccount.md Outdated Show resolved Hide resolved
site/react/useAccount.md Outdated Show resolved Hide resolved
site/react/useSmartAccountClient.md Outdated Show resolved Hide resolved
site/react/useSmartAccountClient.md Show resolved Hide resolved
site/react/useSmartAccountClient.md Outdated Show resolved Hide resolved
site/react/useSmartAccountClient.md Show resolved Hide resolved
Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
@moldy530 moldy530 force-pushed the moldy/add-initial-hooks-docs branch from f4a96db to e1b9664 Compare April 10, 2024 20:50
@moldy530 moldy530 merged commit 5726116 into moldy/prototype-context Apr 10, 2024
2 checks passed
@moldy530 moldy530 deleted the moldy/add-initial-hooks-docs branch April 10, 2024 20:58
moldy530 added a commit that referenced this pull request Apr 10, 2024
* feat: add alchemy accounts context

* style: address PR feedback

Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>

* fix: address a few PR comments

* docs(hooks): add initial react hooks docs (#561)

* docs(hooks): add initial react hooks docs

* refactor: apply suggestions from code review

Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>

---------

Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>

---------

Co-authored-by: Ajay Vasisht <43521356+avasisht23@users.noreply.github.com>
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