Skip to content

Conversation

ppom0
Copy link
Contributor

@ppom0 ppom0 commented Apr 29, 2025

Fix #879

Not quite sure of what I'm doing, as I'm discovering the codebase

@ppom0 ppom0 force-pushed the fix-sqlpage.link-double-quotes branch from d3fa7db to 58c4d05 Compare April 29, 2025 18:09
@lovasoa
Copy link
Collaborator

lovasoa commented Apr 29, 2025

@lovasoa
Copy link
Collaborator

lovasoa commented Apr 29, 2025

val.to_string() encodes the value as json, so if it's a string, it ends up encoded surrounded with "

@lovasoa
Copy link
Collaborator

lovasoa commented May 19, 2025

sorry for the long delay, and thank you ! the fix will be in 0.35 !

@lovasoa lovasoa merged commit 4d98b9c into sqlpage:main May 19, 2025
9 checks passed
@ppom0
Copy link
Contributor Author

ppom0 commented May 22, 2025

Thanks for actually doing the fix!
I had to dig more to be able to do it and didn't find the time.
Thanks :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

sqlpage.link adds percent-encoded double quotes to list parameters.
2 participants