Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Commit

Permalink
Don't tell the user to add account_id after running `wrangler gener…
Browse files Browse the repository at this point in the history
…ate`

Before, wrangler would give this message:
```
> wrangler generate
  Creating project called `worker`...
  Done! New project created /home/jnelson/src/test-project/worker/worker
  🕵️  You can find your zone_id in the right sidebar of a zone's overview tab at https://dash.cloudflare.com
  🕵️  You can copy your account_id below
  +----------------------------+----------------------------------+
  | Account Name               | Account ID                       |
  +----------------------------+----------------------------------+
  | Jyn514@gmail.com's Account | e1706d218241c1230155b4f91b3218af |
  +----------------------------+----------------------------------+
  🕵️  You will need to update the following fields in the created wrangler.toml file before continuing:
  - account_id
```

Adding the account ID is no longer necessary, so don't suggest adding it.
  • Loading branch information
jyn514 committed Jun 28, 2021
1 parent c9aadfd commit 3958957
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/settings/toml/manifest.rs
Original file line number Diff line number Diff line change
Expand Up @@ -414,9 +414,6 @@ impl Manifest {
let account_id_env = env::var("CF_ACCOUNT_ID").is_ok();
let zone_id_env = env::var("CF_ZONE_ID").is_ok();
let mut top_level_fields: Vec<String> = Vec::new();
if !account_id_env {
top_level_fields.push("account_id".to_string());
}
if let Some(kv_namespaces) = &self.kv_namespaces {
for kv_namespace in kv_namespaces {
top_level_fields.push(format!(
Expand Down

0 comments on commit 3958957

Please sign in to comment.