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

Changed from matches to route subcommand matches #1448

Merged
merged 2 commits into from
Jul 14, 2020
Merged

Conversation

jspspike
Copy link
Contributor

Fixes #1216.
Using example wrangler.toml from issue above:

name = "test-route"
type = "javascript"
account_id = ""
workers_dev = true
route = ""
zone_id = ""

[env.production]
account_id = "1516XXXX"
zone_id = "9c08XXXXX"

Before running wrangler route list --env production would result in:
Error: You must specify a zone_id in your configuration file to use 'wrangler route' commands.
Now gets:

Error:  Code 7003: Could not route to /zones/9c08XXXXX/workers/routes, perhaps your object identifier is invalid?
 Code 7000: No route for that URI

@jspspike jspspike added this to the Multi-Route Improvements milestone Jul 13, 2020
@jspspike jspspike requested a review from a team July 13, 2020 22:54
@jspspike jspspike merged commit 602b899 into master Jul 14, 2020
@delete-merged-branch delete-merged-branch bot deleted the josh/route-list-env branch July 14, 2020 15:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

wrangler route list knows nothing about --env
3 participants