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.

Sign up for GitHub

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: updated error message for attempting to deploy a site with a bad… #6884

Merged
merged 8 commits into from
Oct 25, 2024
Merged

fix: updated error message for attempting to deploy a site with a bad… #6884

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
abd9eac
fix: updated error message for attempting to deploy a site with a bad…
dylanspyer Oct 17, 2024
c1556bd
fix: updated error message for deploying with bad auth key to be more…
dylanspyer Oct 18, 2024
d00355e
Merge branch 'main' into oct-17
dylanspyer Oct 19, 2024
9416aa1
Merge branch 'main' into oct-17
dylanspyer Oct 24, 2024
2c1d6bd
Merge branch 'main' into oct-17
dylanspyer Oct 25, 2024
b2613f4
chore: remove `ts-expect-error`s in `get-site`
dylanspyer Oct 25, 2024
0845522
Merge branch 'main' into oct-17
DanielSLew Oct 25, 2024
e0032a0
Merge branch 'main' into oct-17
DanielSLew Oct 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions src/utils/get-site.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { error } from './command-helpers.js'
import { APIError, error } from './command-helpers.js'

// @ts-expect-error TS(7006) FIXME: Parameter 'api' implicitly has an 'any' type.
export const getSiteByName = async (api, siteName) => {
Expand All @@ -12,7 +12,11 @@ export const getSiteByName = async (api, siteName) => {
}

return siteFoundByName
} catch {
error('Site not found. Please rerun "netlify link"')
} catch (error_) {
if ((error_ as APIError).status === 401) {
error(`${(error_ as APIError).message}: could not retrieve site`)
} else {
error('Site not found. Please rerun "netlify link"')
}
}
}
Loading