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.

Sign up for GitHub

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

Jump to bottom

Show warnings in the Query Builder #612

Closed
rappen opened this issue Jan 11, 2022 · 3 comments
Closed

Show warnings in the Query Builder #612

rappen opened this issue Jan 11, 2022 · 3 comments

Comments

@rappen
Copy link
Owner

rappen commented Jan 11, 2022

Make it easier to find fishy queries.

Related to #556 and #609.
@rappen
Copy link
Owner Author

rappen commented Jan 11, 2022

Example:

image

rappen added a commit that referenced this issue Jan 11, 2022
@rappen
Working with #612, a first start!
ef33472
@MarkMpn
Copy link
Collaborator

MarkMpn commented Jan 11, 2022

I like this, I've wanted to add some sort of warnings list for a while. There's various strange things about FetchXML I've come across over time that I could possibly add into the same structure.

@rappen
Copy link
Owner Author

rappen commented Jan 12, 2022

Here you can see the first warning added to validations - it's about to grow...
https://github.com/rappen/FetchXMLBuilder/blob/master/FetchXmlBuilder/AppCode/Validations.cs

But I guess the code is now robust enough to get it to grow now!

rappen added a commit that referenced this issue Jan 12, 2022
@rappen
More validations added #612
f2bf316
rappen added a commit that referenced this issue Jan 15, 2022
@rappen
Add validation for Order #612
93cab57
@rappen rappen closed this as completed in 028cbee Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rappen @MarkMpn