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

Fix all the errors in the react.md #14

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

Conversation

RomThpt
Copy link

@RomThpt RomThpt commented Jan 15, 2025

Changes Introduced:
1. Project Setup Instructions:
• Added steps to create a Next.js project with recommended configurations for TypeScript, Tailwind CSS, and other options.
2. Dependency Installation:
• Provided installation commands for required packages using multiple package managers (npm, pnpm, yarn, bun).
3. Provider Configuration:
• Explained how to configure the Starknet provider with the Cartridge Controller connector, including session policies and RPC setup.
5. App Integration:
• Detailed how to integrate the provider and components into a React application structure.
6. Development Notes:
• Instructions for enabling HTTPS in development via package.json and next.config.ts.

Copy link

vercel bot commented Jan 15, 2025

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

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Jan 20, 2025 9:14pm

src/pages/controller/examples/react.md Outdated Show resolved Hide resolved
src/pages/controller/examples/react.md Outdated Show resolved Hide resolved
src/pages/controller/examples/react.md Outdated Show resolved Hide resolved
src/pages/controller/examples/react.md Show resolved Hide resolved
@tarrencev
Copy link
Contributor

Thanks for the contribution! Left a few comments

…nore

- Added support for Vite in the React integration guide.
- Updated the button components to use standard HTML buttons instead of Chakra UI's Button.
- Included instructions for enabling HTTPS in development for both Next.js and Vite.
- Added new entries to .gitignore for cursor-related files.
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