added null check to window

[adtjha]

What kind of change does this PR introduce?

Original
if (!url) url = window.location.href

Added null checks to window,
if (!url) url = window.location && window.location.href || ""

What is the current behavior?

Bug : TypeError: undefined is not an object (evaluating 'window.location.href')

react-native : 0.64.2,
expo : ~43.0.0,
@supabase/supabase-js : ^1.25.2,

TypeError: undefined is not an object (evaluating 'window.location.href')
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:104:6 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:172:19 in handleException
at node_modules/react-native/Libraries/Core/setUpErrorHandling.js:24:6 in handleError
at node_modules/@react-native/polyfills/error-guard.js:49:36 in ErrorUtils.reportFatalError
at node_modules/metro-runtime/src/polyfills/require.js:204:6 in guardedLoadModule
at http://192.168.1.37:19000/index.bundle?platform=android&dev=true&hot=false&minify=false:237331:3 in global code

Invariant Violation: "main" has not been registered. This can happen if:
* Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
* A module failed to load due to an error and `AppRegistry.registerComponent` wasn't called.
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:104:6 in reportException
at node_modules/react-native/Libraries/Core/ExceptionsManager.js:172:19 in handleException
at node_modules/react-native/Libraries/Core/setUpErrorHandling.js:24:6 in handleError
at node_modules/@react-native/polyfills/error-guard.js:49:36 in ErrorUtils.reportFatalError

What is the new behavior?

No more TypeError, works perfectly fine.

[kiwicopple]

@adtjha could you please merge the latest changes from master, resolve any conflicts, and then run npm run format? Otherwise this looks good!

FYI, I think this should work too: window?.location?.href || ''

[kiwicopple]

fixes supabase/supabase#3790

[Seanmclem]

I see this is merged, but I'm still facing this issue with @supabase/supabase-js 1.27.0. How can I make sure I get the fix? ... Temporarily made the fix in my node modules directory. But it seems like supabase needs updated to the latest gotrue-js version? I'd make the PR myself, but I can't find where the version number changes

[github-actions]

🎉 This PR is included in version 1.21.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀