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

[Linting & Validation] expression fields in arrays do not get validated #1783

Closed
a-b-r-o-w-n opened this issue Dec 17, 2019 · 2 comments
Closed

[Linting & Validation] expression fields in arrays do not get validated #1783

a-b-r-o-w-n opened this issue Dec 17, 2019 · 2 comments
Assignees
@lei9444
Labels
Area: Form Editor R8 Release 8 - March 16th, 2020 Type: Bug Something isn't working

Comments

@a-b-r-o-w-n
Copy link
Contributor

For instance, DeleteProperties.items.property does not get validated as an expression.
@a-b-r-o-w-n a-b-r-o-w-n added Type: Bug Something isn't working Area: Form Editor labels Dec 17, 2019
@a-b-r-o-w-n a-b-r-o-w-n assigned tdurnford and unassigned tdurnford Dec 17, 2019
@a-b-r-o-w-n a-b-r-o-w-n changed the title [Form Editor] expression fields in arrays do not get validated [Linting & Validation] expression fields in arrays do not get validated Dec 17, 2019
@a-b-r-o-w-n
Copy link
Contributor Author

Upon some investigation, the function getExpressionProperties does not get all of the properties that need to be validated as expressions so these are not getting picked up.

@lei9444 lei9444 mentioned this issue Dec 23, 2019
Merged
9 tasks
@hibrenda hibrenda added the R8 Release 8 - March 16th, 2020 label Dec 30, 2019
@cwhitten
Copy link
Member

cwhitten commented Jan 6, 2020

addressed via #1798

@cwhitten cwhitten closed this as completed Jan 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lei9444 lei9444

Labels
Area: Form Editor R8 Release 8 - March 16th, 2020 Type: Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@cwhitten @a-b-r-o-w-n @tdurnford @lei9444 @hibrenda