Skip to content
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

How to override or format group query field and value? #912

Open
NikunjPansuriya opened this issue May 7, 2023 · 7 comments · May be fixed by #913
Open

How to override or format group query field and value? #912

NikunjPansuriya opened this issue May 7, 2023 · 7 comments · May be fixed by #913
Labels
Milestone

Comments

@NikunjPansuriya
Copy link
Contributor

NikunjPansuriya commented May 7, 2023

Is it possible to generate rule like below format?

any(results, {.product == 'abc' && .score > 8}) or
any(results, {.product == 'abc' || .score > 8})

@ukrbublik
Copy link
Owner

I need to make some code changes to make this format possible.
Is this a specific language?

@NikunjPansuriya
Copy link
Contributor Author

No, its not a specific language. its an expressions. You can check more details regarding expressions here.

@NikunjPansuriya
Copy link
Contributor Author

are you going to make changes for given expression rule?

@ukrbublik
Copy link
Owner

Yes

ukrbublik added a commit that referenced this issue May 9, 2023
ukrbublik added a commit that referenced this issue May 9, 2023
This reverts commit 62289bd.
ukrbublik added a commit that referenced this issue May 9, 2023
* added next.js app

* .

* .

* sandbox_simple - remove complex config

* todo: why complex ?

* doc

* .

* .

* .

* .

* lint fix

* reuse funcs

* serialize export funcs

* render from context

* tmp!!!!

* Revert "tmp!!!!"

This reverts commit 290487a.

* fix sanitizeTree

* .

* wip

* .

* .

* .

* .

* ctx  better shape

* ssr antd

* all components converted to use ctx

* todo: session in cookie is bad

* sid

* test app

* config mixins

* .

* lint fix

* cs ci

* tests fix

* jsx fix

* cs ci fix

* new session approach

* .
.

* .

* .

* patch version

* body len

* node-fetch

* throttle

* Revert "node-fetch"

This reverts commit 6990f0b.

* redis vs file

* lint fix

* redis

* .

* 6.2.0

* readme

* fix onChange 2nd param - return basic config, not extended

* fix compress for []

* readme

* .

* .

* typo

* .

* .

* .

* .

* not working

* sanitizeTree?

* ?

* .

* readme

* tests

* tests

* fix types

* tests

* delete @mui/lab

* applyJsonLogic

* tree funcs

* fixed & added tests

* more tests

* typo

* try to fix
fix time
test1
test2
test3
test4
test5
test6
upd test packages
codecov for pr
.
node 16
codecov
log
.
test
.
enable cache
FIX TESTS
SKIP TEST
Revert "test1"

This reverts commit cdb9a35d0b62c5fa6459b95d2b38357825fa5e51.

* throw

* cov

* .

* ?

* .

* .

* log

* .

* 1111

* clean time

* clean

* lint fix

* upd @mui

* .

* wip #912 todo

* Revert "wip #912 todo"

This reverts commit 62289bd.

* test: can use reversed op

* more doc

* .

* .

* urls

* .
.
.
.
.

* .

* .

* .

* .
ukrbublik added a commit that referenced this issue May 9, 2023
@NikunjPansuriya
Copy link
Contributor Author

@ukrbublik
Any update on expression rule support?
Is PR merged for given rule support?

@ukrbublik
Copy link
Owner

No updates yet. My changes are WIP.
I will work on #913 after merging #900

ukrbublik added a commit that referenced this issue Jul 5, 2023
ukrbublik added a commit that referenced this issue Jul 21, 2023
@ukrbublik ukrbublik linked a pull request Jul 21, 2023 that will close this issue
@ukrbublik ukrbublik added this to the Export milestone Jul 27, 2023
@NikunjPansuriya
Copy link
Contributor Author

@ukrbublik
when you are planning to merge PR? if it take time then how i can install specific package for e.g. MUI with given PR?

ukrbublik added a commit that referenced this issue May 23, 2024
ukrbublik added a commit that referenced this issue May 23, 2024
ukrbublik added a commit that referenced this issue May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants