bbende commented on pull request #4512:
URL: https://github.com/apache/nifi/pull/4512#issuecomment-700250454
This is an awesome feature! I'm loving the simplification of the record
reader properties :)
I came across one issue while testing this which looks like a UI issue. I
took the test processor from this PR that is named `DependsOnProperties` and I
put it in a NAR and deployed it to my NiFi. I added an instance of it to the
canvas and it was correctly invalid, but when I tried to open the configuration
to change the properties I got this error in the browser console:
```
jquery.propertytable.js?1.13.0-SNAPSHOT:1 Uncaught TypeError: Cannot read
property 'includes' of undefined
at Object.<anonymous> (jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at Function.each (jquery.min.js:2)
at Function.b.each (jquery.each.js:1)
at Object.<anonymous> (jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at Function.each (jquery.min.js:2)
at Function.b.each (jquery.each.js:1)
at jquery.propertytable.js?1.13.0-SNAPSHOT:1
at Function.each (jquery.min.js:2)
at Function.b.each (jquery.each.js:1)
at D (jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at HTMLDivElement.<anonymous> (jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at Function.each (jquery.min.js:2)
at Function.b.each (jquery.each.js:1)
at S.fn.init.each (jquery.min.js:2)
at S.fn.init.loadProperties (jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at S.fn.init.t.fn.propertytable
(jquery.propertytable.js?1.13.0-SNAPSHOT:1)
at Array.<anonymous> (nf-canvas-all.js?1.13.0-SNAPSHOT:25)
at c (jquery.min.js:2)
at Object.fireWith [as resolveWith] (jquery.min.js:2)
at Object.<anonymous> (jquery.min.js:2)
at c (jquery.min.js:2)
at Object.fireWith [as resolveWith] (jquery.min.js:2)
at l (jquery.min.js:2)
at XMLHttpRequest.<anonymous> (jquery.min.js:2)
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
