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

add getDetailsString() #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

symbiogenesis
Copy link

@symbiogenesis symbiogenesis commented Oct 9, 2020

This allows you to see the details in a human-readable syntax. Recursively handles all groupings.

Something like:

((fieldOne = 250) AND (fieldTwo > 3)) OR (fieldThree BETWEEN 1 AND 100))

@symbiogenesis
Copy link
Author

@SandyDatatables @colin0117 do you think this could get merged?

@SandyDatatables
Copy link
Member

Hi @symbiogenesis ,

Just wondering what the use case is for this? Are you displaying it as a string to the user or something along those lines?

Thanks,
Sandy

@symbiogenesis
Copy link
Author

Yes, I'm displaying it to the user when they export to a file or print. I thought others might find this code useful.

@SandyDatatables
Copy link
Member

That's a really nice idea. However we do have a couple of concerns about internationalisation and how much use this would actually get.

What we will do is leave it here to allow for feedback from others. If there is enough interest then we will pull it in at that point.

Thanks,
Sandy

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.

2 participants