benjypng / chrome-extension-logseq-quickcapture

chrome-extension-logseq-quickcapture

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

URL-only captures include `null` before URL

sarahec opened this issue · comments

Version 0.2.5

To reproduce

  1. Open a web page
  2. Do not make a text selection
  3. Click the capture extension
  4. Observe the newly-added null.
**14:43** [[quick capture]]: null [Youtube Video RAG Search - crewAI](https://docs.crewai.com/tools/YoutubeVideoSearchTool/)

Cause

Ifresult is null, don't include it in the content URL parameter.

Apologies for the multiple updates, I had to dig to find the breakage. I had tested with and without a text selection (which worked), but forgot to url encode the result. So ASCII content worked, but apparently non-ASCII didn't.

Thanks for reporting this! 0.2.6 is pending publication in the chrome store.

0.2.6 is published.