Skip to content

Commit c34ec83

Browse files
committed
Added comment for discoverAltOrigin
* Also included up no longer used import
1 parent 0e6b512 commit c34ec83

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/managers/AltOriginManager.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { AppConfig } from '../models/AppConfig';
22
import { EnvironmentKind } from '../models/EnvironmentKind';
33
import ProxyFrameHost from '../modules/frames/ProxyFrameHost';
4-
import { contains } from '../utils';
54
import SdkEnvironment from './SdkEnvironment';
65

76
export default class AltOriginManager {
@@ -10,7 +9,13 @@ export default class AltOriginManager {
109

1110
}
1211

13-
static async discoverAltOrigin(appConfig): Promise<ProxyFrameHost> {
12+
/*
13+
* This loads all possible iframes that a site could be subscribed to
14+
* (os.tc & onesignal.com) then checks to see we are subscribed to any.
15+
* If we find what we are subscribed to both unsubscribe from onesignal.com.
16+
* This method prefers os.tc over onesignal.com where possible.
17+
*/
18+
static async discoverAltOrigin(appConfig: AppConfig): Promise<ProxyFrameHost> {
1419
const iframeUrls = AltOriginManager.getOneSignalProxyIframeUrls(appConfig);
1520

1621
const allProxyFrameHosts: ProxyFrameHost[] = [];

0 commit comments

Comments
 (0)