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

Adds helpful error message on authentication error #932

Merged
merged 2 commits into from
Dec 6, 2019
Merged

Adds helpful error message on authentication error #932

merged 2 commits into from
Dec 6, 2019

Conversation

EverlastingBugstopper
Copy link
Contributor

Fixes #492

Before

$ wrangler publish
💁  JavaScript project found. Skipping unnecessary build!
Error: Something went wrong! Status: 400 Bad Request, Details {"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}

After

$ wrangler publish
💁  JavaScript project found. Skipping unnecessary build!
Error: Your user configuration is invalid, please run wrangler config and enter a new set of credentials.

@EverlastingBugstopper EverlastingBugstopper added this to the 1.7.0 milestone Dec 6, 2019
@EverlastingBugstopper EverlastingBugstopper requested a review from a team December 6, 2019 17:52
@EverlastingBugstopper EverlastingBugstopper self-assigned this Dec 6, 2019
@ashleygwilliams ashleygwilliams merged commit 615e1b5 into master Dec 6, 2019
@ashleygwilliams ashleygwilliams deleted the avery/auth-auth-auth branch December 6, 2019 18:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ashleygwilliams ashleygwilliams ashleygwilliams approved these changes

Assignees

@EverlastingBugstopper EverlastingBugstopper

Labels
None yet
Projects
None yet
Milestone
1.7.0
Development

Successfully merging this pull request may close these issues.

Fix messaging for invalid credentials w/authenticated preview
2 participants
@EverlastingBugstopper @ashleygwilliams