URL-only captures include `null` before URL
sarahec opened this issue · comments
Sarah Clark commented
Version 0.2.5
To reproduce
- Open a web page
- Do not make a text selection
- Click the capture extension
- 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.
Sarah Clark commented
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.
benjypng commented
Thanks for reporting this! 0.2.6 is pending publication in the chrome store.
benjypng commented
0.2.6 is published.