Skip to content

fix(docs): update step 11 to reflect .env usage#17

Open
stackoverflowed1 wants to merge 1 commit into
coinbase:mainfrom
stackoverflowed1:stackoverflowed1-patch-1
Open

fix(docs): update step 11 to reflect .env usage#17
stackoverflowed1 wants to merge 1 commit into
coinbase:mainfrom
stackoverflowed1:stackoverflowed1-patch-1

Conversation

@stackoverflowed1
Copy link
Copy Markdown

Description

Step 11 in the README previously instructed users to "Fill in CDP project id in App.tsx (replace REPLACE_ME with the project id)". This is outdated and misleading, as the current App.tsx reads Config.CDP_PROJECT_ID from the .env file and throws an error if it is missing — there is no REPLACE_ME in the code.

To clarify, step 11 has been updated to instruct users to ensure CDP_PROJECT_ID is set in .env (referencing step 3). This keeps the guide user-friendly for newcomers while removing the inaccurate instruction, without deleting the step entirely.

Step 11 in the README previously instructed users to "Fill in CDP project id in `App.tsx` (replace `REPLACE_ME` with the project id)". 
This is outdated and misleading, as the current App.tsx reads Config.CDP_PROJECT_ID from the .env file and throws an error if it is missing — there is no REPLACE_ME in the code. 

To clarify, step 11 has been updated to instruct users to ensure CDP_PROJECT_ID is set in `.env` (referencing step 3). 
This keeps the guide user-friendly for newcomers while removing the inaccurate instruction, without deleting the step entirely.
@cb-heimdall
Copy link
Copy Markdown

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants