RoamJS / query-builder

https://roamjs.com/extensions/query-builder
MIT License
20 stars 5 forks source link

Extension will not load in iOS #102

Open DaveWelling opened 1 year ago

DaveWelling commented 1 year ago

I've tried clearing the cache and loading in both iOS safari and chrome. I tried updating both Roam and the Roam Depot extensions. Error in load-extension for extension: Query Builder Line: 60 sourceURL: blob:https://roamresearch.com/e29c9ef5-e649-4c28-8c6c-9e76884cd871

dvargas92495 commented 1 year ago

If you enable developer mode, and you load an extension via URL, does entering https://samepage.network/releases/dvargas92495/roamjs-query-builder load?

DaveWelling commented 1 year ago

Hi @dvargas92495 , thanks so much for replying. I didn't expect that on a Saturday. Here's what I get when (I think) follow your instructions: 5DE5C7FA-A2CF-466A-80EB-8BAF853ACD97

DaveWelling commented 1 year ago

BTW, I also tried to perform the same operation in the Inspect Browser so I could give you console information: Screenshot 2023-06-17 at 12 53 13 PM

dvargas92495 commented 1 year ago

I added some error reporting so that it emails some extra context on why it could be failing. Could you try again? I expect it to fail but that it will send me more info as to why

DaveWelling commented 1 year ago

Sure David, Just turned off the browser cache and retried loading it, hopefully that got your new code. Let me know if it didn't work, and I can blow away the whole cache.

baibhavbista commented 1 year ago

Hey @dvargas92495 , btw I'm getting CORS issues on "https://samepage.network/releases/dvargas92495/roamjs-query-builder" when using dev extensions via URL. Please check my screenshot below: image

@DaveWelling, in the dev extensions by URL, can you try "dvargas92495+query-builder+695" instead? and let us know if you see different errors

(I'm from the roam team btw, wanted to let you know that I'm not a rando asking you to load some arbitrary code in your graph 😅. In general, you should be really careful about loading extensions via the developer urls since those are not reviewed by us)

DaveWelling commented 1 year ago

@baibhavbista I don't have my iPad in front of me right now. When I have it again, should I just put dvargas92495+query-builder+695 as the entire URL? I'm guessing it will preface that with a roam URL to avoid the CORS?