send-to-workflowy icon indicating copy to clipboard operation
send-to-workflowy copied to clipboard

Bookmarklet fails to send

Open jeffreywyman opened this issue 3 years ago • 2 comments

Some pages fail to send WF bookmark, seen on github.com and google.com addresses.

`Content Security Policy: The page’s settings blocked the loading of a resource at https://url-hidden.netlify.app/send (“connect-src”). javascript:1:232

Uncaught (in promise) TypeError: NetworkError when attempting to fetch resource.`

If this cannot be fixed, it would be good to show a toast notification for success/failure.

jeffreywyman avatar Feb 23 '22 02:02 jeffreywyman

Yeah, I think the bookmarklet code could do with an overhaul. I was going to take some inspiration from linkding and/or instapaper. Stretch goal would be a web extension instead.

Can you give me an example of a page for which it errors for you? Is this on Chrome?

cjlm avatar Feb 24 '22 01:02 cjlm

This is on Firefox, here's the console when I use it on github (for example this repo) [image: image.png]

On Wed, Feb 23, 2022 at 8:31 PM cjlm @.***> wrote:

Yeah, I think the bookmarklet code could do with an overhaul. I was going to take some inspiration from linkding and/or instapaper. Stretch goal would be a web extension instead.

Can you give me an example of a page for which it errors for you? Is this on Chrome?

— Reply to this email directly, view it on GitHub https://github.com/cjlm/send-to-workflowy/issues/12#issuecomment-1049405002, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABF5CWZARDHITRGGRGRKNM3U4WC6HANCNFSM5PDDD3ZQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.***>

jeffreywyman avatar Feb 24 '22 01:02 jeffreywyman