Skip to content

Commit 19bd482

Browse files
committed
hotfix(urls): Partially reverts the previous change for setting URLs in code samples. Fixes a missing variable.
1 parent 96ddfba commit 19bd482

File tree

2 files changed

+21
-20
lines changed

2 files changed

+21
-20
lines changed

assets/js/influxdb-url.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,11 @@ import { context as PRODUCT_CONTEXT, referrerHost } from './page-context.js';
1818
import { delay } from './helpers.js';
1919
import { toggleModal } from './modals.js';
2020

21-
export const CLOUD_URLS = Object.values(pageParams.influxdb_urls.cloud.providers).flatMap((provider) => provider.regions?.map((region) => region.url));
21+
let CLOUD_URLS = [];
22+
if (pageParams && pageParams.influxdb_urls) {
23+
CLOUD_URLS = Object.values(pageParams.influxdb_urls.cloud.providers).flatMap((provider) => provider.regions?.map((region) => region.url));
24+
}
25+
export { CLOUD_URLS };
2226

2327
export function InfluxDBUrl() {
2428
const UNIQUE_URL_PRODUCTS = ['dedicated', 'clustered'];
@@ -577,7 +581,7 @@ export function InfluxDBUrl() {
577581
removeCustomUrl();
578582
hideValidationMessage();
579583
$(
580-
`input[name="influxdb-${PRODUCT_CONTEXT}-url"][value="${DEFAULT_URLS[PRODUCT_CONTEXT]}"]`
584+
`input[name="influxdb-${PRODUCT_CONTEXT}-url"][value="${DEFAULT_STORAGE_URLS[PRODUCT_CONTEXT]}"]`
581585
).trigger('click');
582586
}
583587
}

assets/js/local-storage.js

Lines changed: 15 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -87,28 +87,25 @@ const defaultUrls = {};
8787
// Guard against pageParams being null/undefined and safely access nested properties
8888
if (pageParams && pageParams.influxdb_urls) {
8989
Object.entries(pageParams.influxdb_urls).forEach(([product, {providers}]) => {
90-
// Use optional chaining and nullish coalescing to safely access properties
91-
const defaultProvider = providers?.filter(provider => provider?.name === 'Default')[0];
92-
const defaultRegion = defaultProvider?.regions?.[0];
93-
defaultUrls[product] = defaultRegion?.url || '';
90+
defaultUrls[product] = providers.filter(provider => provider.name === 'Default')[0]?.regions[0]?.url;
9491
});
9592
}
9693

9794
export const DEFAULT_STORAGE_URLS = {
98-
oss: defaultUrls.oss || '',
99-
cloud: defaultUrls.cloud || '',
100-
serverless: defaultUrls.serverless || '',
101-
core: defaultUrls.core || '',
102-
enterprise: defaultUrls.enterprise || '',
103-
dedicated: defaultUrls.cloud_dedicated || '',
104-
clustered: defaultUrls.clustered || '',
105-
prev_oss: defaultUrls.oss || '',
106-
prev_cloud: defaultUrls.cloud || '',
107-
prev_core: defaultUrls.core || '',
108-
prev_enterprise: defaultUrls.enterprise || '',
109-
prev_serverless: defaultUrls.serverless || '',
110-
prev_dedicated: defaultUrls.cloud_dedicated || '',
111-
prev_clustered: defaultUrls.clustered || '',
95+
oss: defaultUrls.oss,
96+
cloud: defaultUrls.cloud,
97+
serverless: defaultUrls.serverless,
98+
core: defaultUrls.core,
99+
enterprise: defaultUrls.enterprise,
100+
dedicated: defaultUrls.cloud_dedicated,
101+
clustered: defaultUrls.clustered,
102+
prev_oss: defaultUrls.oss,
103+
prev_cloud: defaultUrls.cloud,
104+
prev_core: defaultUrls.core,
105+
prev_enterprise: defaultUrls.enterprise,
106+
prev_serverless: defaultUrls.serverless,
107+
prev_dedicated: defaultUrls.cloud_dedicated,
108+
prev_clustered: defaultUrls.clustered,
112109
custom: '',
113110
};
114111

0 commit comments

Comments
 (0)