Unexpected error only on some computers about hull HOT 5 CLOSED

Hey there @j4ysen!

I'm guessing the storeURL is not set properly in Sanity, and therefore it's failing at this line: https://github.com/ndimatteo/HULL/blob/main/components/cart.js#L52-L53

That's the only place where we use that variable, so if you can run some checks there, might provide some insight!

from hull.

Thanks for the quick reply Nick! I've done some checks and there's no reason why that would be undefined (since everything is properly defined in sanity). It works fine on new browsers, but on safari 12 for example it doesn't work. I'll do some more research and post here if I find out anything new.

from hull.

Did some research on a virtual machine of safari 12, and it seems that the 'data' put into cart comes up as empty

Is there any reason this might be the case on certain browsers?
Also seeing a lot of 404s, especially on the build manifest...

from hull.

I'm wondering if this is a bug with Safari 12, perhaps with how we destructure the data object for the Cart component here: https://github.com/ndimatteo/HULL/blob/main/components/cart.js#L19-L20

Can you try adding:

if (!data) return null

...above the destructuring to see if that helps?

from hull.

Closing due to inactivity, feel free to re-open if you have more details about the issue 🤘

from hull.