-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: fix config changes filter by cell issues around encoding and colons #2012
fix: fix config changes filter by cell issues around encoding and colons #2012
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for clerk-saas-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for goofy-euclid-75956c ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for flanksource-demo-stable ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
d2f2015
to
d08c018
Compare
d08c018
to
fdeec1a
Compare
fdeec1a
to
47a5a47
Compare
47a5a47
to
3f5c1fd
Compare
@mainawycliffe I think the values need to be urlencoded, then combined with a |
7e4b97d
to
0fafc19
Compare
@moshloop @adityathebe what about instead of a GET request, we use post and body for the filters for this endpoint? This would eliminate the need for double encoding. |
0c7d8ff
to
8d0f45b
Compare
8d0f45b
to
1491d2d
Compare
1491d2d
to
fd48a4a
Compare
fd48a4a
to
1dd909f
Compare
1dd909f
to
a9d6d22
Compare
Fixes #1970 fix: switch to post for catalog changes, instead of get fix: envode value and symbol fix: incorrect encoding issues
a9d6d22
to
e0afae5
Compare
Fixes #1970
Fixes #2017