James Hill comments

Results 22 comments of


                                            James Hill

Direct localStorage/ sessionStorage access prevents usage from Worker context or Safari Private tab

Hey @Tombarr, This is great work! Let me know if you're still having issues. I'll Pull your branch and take a look if so 👌

Direct localStorage/ sessionStorage access prevents usage from Worker context or Safari Private tab

Hey @Tombarr, Great! I'm travelling for work over the next few days but I'll try and have a thorough look when I get time / when I'm back. I've had...

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Hey @Jeroen999, Thanks for reaching out. Have you had a read of the supplied error docs in your snippet above? If that means something to you, that would be a...

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Yes, you'll be able to track conversions based on page location, path, etc. You can also manually track conversion via custom events, if needed. The distributable is provided in the...

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Thanks @Jeroen999, that would give me something to work with. Before you do so, could you copy the _actual_ code that's being included on the page, e.g from the document?...

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Hmm, ok thanks. Do you have a public url I can take a look at?

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

OK great, can you DM me on Twitter (I don't know if Github has an equivalent, and I don't really want to publicly post): https://twitter.com/jahilldev

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Sorry, DM now open

require.min.js:17 Uncaught Error: Mismatched anonymous define() module:

Hey @Jeroen999, apologies on the delay, I'm unable to find time to look at this right now, I'll try and get back to it later in the week 👍

Check if `navigator.sendBeacon` is defined before calling it

Hey @PascalKoe, Yes this should be a fairly simple change, we'd just need to fall back to making a fetch request instead. The only consideration being `sendBeacon` ensures to request...