Skip to content

Update 'getting started' guide #392

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

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

Conversation

jrh3k5
Copy link

@jrh3k5 jrh3k5 commented Feb 1, 2025

While trying to get started with Farcaster v2 Frames, I encountered a few issues:

  • I didn't realize I wasn't supposed to use the default option for the prompt of import aliasing; I added doc to call that out explicitly.
  • The yarn command doesn't automatically add next as an executable; I had to use yarn add... to get it added to the project; I've added doc calling that out explicitly.
  • One of the example Demo files is missing an import and function declaration, which prevented compilation; I've added that.
  • The app/globals.css file referenced at one point doesn't actually exist in the project by that point, so I've added a note about needing to ensure that file exists.

PR-Codex overview

This PR enhances the documentation for setting up a NextJS app with frames-v2. It includes additional instructions for project setup, dependency installation, and modifications to the app layout and components.

Detailed summary

  • Updated instructions for setting up a NextJS app with frames-v2-demo.
  • Added command to install next, react, and react-dom.
  • Instructed to create app/globals.css if it doesn't exist.
  • Introduced isContextOpen state in the Demo component.
  • Added toggleContext function to manage isContextOpen state.

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

Copy link

vercel bot commented Feb 1, 2025

@jrh3k5 is attempting to deploy a commit to the farcaster Team on Vercel.

A member of the Team first needs to authorize it.

@jrh3k5 jrh3k5 marked this pull request as ready for review February 1, 2025 21:05
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