Skip to content

Commit

Permalink
feat(config): Add console.warn for depricated params.
Browse files Browse the repository at this point in the history
This includes interfaceConfig.SUPPORT_URL, interfaceConfig.LIVE_STREAMING_HELP_LINK, config.defaultLogoUrl, config.deploymentUrls, config.liveStreaming and config.customToolbarButtons.
  • Loading branch information
hristoterezov committed Feb 24, 2025
1 parent 8027b1d commit 9db4f42
Showing 1 changed file with 43 additions and 0 deletions.
43 changes: 43 additions & 0 deletions react/features/base/config/functions.any.ts
Original file line number Diff line number Diff line change
Expand Up @@ -327,6 +327,49 @@ export function setConfigFromURLParams(
}

overrideConfigJSON(config, interfaceConfig, json);

// Print warning about depricated URL params
if ('interfaceConfig.SUPPORT_URL' in params) {
console.warn('Using SUPPORT_URL interfaceConfig URL overwrite is deprecated.'
+ ' Please use supportUrl from advanced branding!');
}

if ('config.defaultLogoUrl' in params) {
console.warn('Using defaultLogoUrl config URL overwrite is deprecated.'
+ ' Please use defaultLogoUrl from advanced branding!');
}

const deploymentUrlsConfig = params['config.deploymentUrls'] ?? {};

if ('config.deploymentUrls.downloadAppsUrl' in params || 'config.deploymentUrls.userDocumentationURL' in params
|| (typeof deploymentUrlsConfig === 'object'
&& ('downloadAppsUrl' in deploymentUrlsConfig || 'userDocumentationURL' in deploymentUrlsConfig))) {
console.warn('Using deploymentUrls config URL overwrite is deprecated.'
+ ' Please use downloadAppsUrl and/or userDocumentationURL from advanced branding!');
}

const liveStreamingConfig = params['config.liveStreaming'] ?? {};

if (('interfaceConfig.LIVE_STREAMING_HELP_LINK' in params)
|| ('config.liveStreaming.termsLink' in params)
|| ('config.liveStreaming.dataPrivacyLink' in params)
|| ('config.liveStreaming.helpLink' in params)
|| (typeof params['config.liveStreaming'] === 'object' && 'config.liveStreaming' in params
&& (
'termsLink' in liveStreamingConfig
|| 'dataPrivacyLink' in liveStreamingConfig
|| 'helpLink' in liveStreamingConfig
)
)) {
console.warn('Using liveStreaming config URL overwrite and/or LIVE_STREAMING_HELP_LINK interfaceConfig URL'
+ ' overwrite is deprecated. Please use liveStreaming from advanced branding!');
}

if ('config.customToolbarButtons' in params) {
console.warn('Using customToolbarButtons config URL overwrite is deprecated.'
+ ' Please use liveStreaming from advanced branding!');
}

}

/* eslint-enable max-params */
Expand Down

0 comments on commit 9db4f42

Please sign in to comment.