Skip to content

Commit 4c1bd42

Browse files
joehanhlshen
andauthored
Switching a few more places to getters (#6914)
Co-authored-by: Harold Shen <hlshen@google.com>
1 parent 6950829 commit 4c1bd42

File tree

4 files changed

+6
-9
lines changed

4 files changed

+6
-9
lines changed

src/api.ts

-5
Original file line numberDiff line numberDiff line change
@@ -35,11 +35,6 @@ export const authOrigin = () =>
3535
utils.envOverride("FIREBASE_AUTH_URL", "https://accounts.google.com");
3636
export const consoleOrigin = () =>
3737
utils.envOverride("FIREBASE_CONSOLE_URL", "https://console.firebase.google.com");
38-
export const deployOrigin = () =>
39-
utils.envOverride(
40-
"FIREBASE_DEPLOY_URL",
41-
utils.envOverride("FIREBASE_UPLOAD_URL", "https://deploy.firebase.com"),
42-
);
4338
export const dynamicLinksOrigin = () =>
4439
utils.envOverride("FIREBASE_DYNAMIC_LINKS_URL", "https://firebasedynamiclinks.googleapis.com");
4540
export const dynamicLinksKey = () =>

src/gcp/devConnect.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -254,5 +254,5 @@ export async function getGitRepositoryLink(
254254
* Returns email associated with the Developer Connect Service Agent
255255
*/
256256
export function serviceAgentEmail(projectNumber: string): string {
257-
return `service-${projectNumber}@${developerConnectP4SAOrigin}`;
257+
return `service-${projectNumber}@${developerConnectP4SAOrigin()}`;
258258
}

src/gcp/identityPlatform.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import { Client } from "../apiv2";
55
const API_VERSION = "v2";
66

77
const adminApiClient = new Client({
8-
urlPrefix: identityOrigin + "/admin",
8+
urlPrefix: identityOrigin() + "/admin",
99
apiVersion: API_VERSION,
1010
});
1111

src/init/features/hosting/github.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ export async function initGitHub(setup: Setup): Promise<void> {
206206
`Visit this URL to revoke authorization for the Firebase CLI GitHub OAuth App:`,
207207
);
208208
logger.info(
209-
bold(underline(`https://github.com/settings/connections/applications/${githubClientId}`)),
209+
bold(underline(`https://github.com/settings/connections/applications/${githubClientId()}`)),
210210
);
211211
logLabeledBullet("Action required", `Push any new workflow file(s) to your repo`);
212212
}
@@ -442,7 +442,9 @@ async function promptForRepo(
442442
);
443443
logger.info(
444444
bold(
445-
underline(`https://github.com/settings/connections/applications/${githubClientId}`),
445+
underline(
446+
`https://github.com/settings/connections/applications/${githubClientId()}`,
447+
),
446448
),
447449
);
448450
logger.info();

0 commit comments

Comments
 (0)