mcgilman commented on issue #3606: [WIP] Nifi 6282 URL: https://github.com/apache/nifi/pull/3606#issuecomment-526314519 @scottyaslan Thanks for the PR! Lot's good stuff here. This has been merged to master. One issue I found that we should address in the future... If the user does not have read access to the Process Group and they attempt to update a parameter context, they can access the Parameters context menu item if we are in the group in question. However, they cannot access the Parameters context menu item if they are in the parent group and select the group in question on the canvas. We likely need to update the back end to include the parameter context id in the process group entity. This is something that can be tracked in a follow on effort. @rfellows Thanks for the review!
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services