Adding better error messaging when using non Firebase projects #8021
Conversation
src/utils.ts
Outdated
| /** | ||
| * Create a Migrate to Firebase Console URL for the specified project. | ||
| */ | ||
| export function migrateToFirebaseLink(project: string): string { |
There was a problem hiding this comment.
Do you have flexibility on this verb "migrate"? It's not really a "migrate to Firebase" but more like "add Firebase" if that's possible?
There was a problem hiding this comment.
Definitely - I took 'migrate' from the query parameter in the console link, but if 'add Firebase' is more accurate lets use that
There was a problem hiding this comment.
ya, from the dev perspective, it's messaged as "add Firebase"
e.g., https://firebase.google.com/docs/reference/firebase-management/rest/v1beta1/projects/addFirebase
Co-authored-by: rachelsaunders <[email protected]>
Co-authored-by: rachelsaunders <[email protected]>
src/utils.ts
Outdated
| /** | ||
| * Create a Migrate to Firebase Console URL for the specified project. | ||
| */ | ||
| export function migrateToFirebaseLink(project: string): string { |
There was a problem hiding this comment.
ya, from the dev perspective, it's messaged as "add Firebase"
e.g., https://firebase.google.com/docs/reference/firebase-management/rest/v1beta1/projects/addFirebase
Description
Adding better error messaging when using projects that do not have Firebase enabled.