Github user kirkalicious commented on the issue:
https://github.com/apache/nifi/pull/969
If the DynamicProperty annotation is absent, I can check if the
getSupportedDynamicPropertyDescriptor method is defined in
AbstractConfigurableComponent. I'll hide the '+' if the annotation is
present or if the method is overridden. Thoughts?
On Tuesday, August 30, 2016, Peter <[email protected]
<javascript:_e(%7B%7D,'cvml','[email protected]');>> wrote:
> I would love to just show/hide it, but I can see where people are coming
> from.
> What if for v1.1 an outright show/hide was replaced with an alternative
> color, maybe make the button look like it's disabled while leaving it
> enabled by applying a CSS class. This would at least give a visual
> indication to users without breaking anything for custom processors.
>
> Try to let users know what's going to happen, get other processors
> updated, then in v1.x you can start actually showing/hiding?
>
> â
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/nifi/pull/969#issuecomment-243648647>, or mute
> the thread
>
<https://github.com/notifications/unsubscribe-auth/ABcVqBOpHslqrKTjYHczwCaGPbd0x0RZks5qlPVGgaJpZM4JwNlu>
> .
>
--
Kirk Tarou
The 9:20 Special - Lindy Hop & Swing Dance Venue
2460 Sutter Street, San Francisco, CA, 94115
www.920special.com
Find us here: Facebook <http://www.facebook.com/group.php?gid=8134453869>
Twitter <http://twitter.com/920special> Yelp!
<http://www.yelp.com/biz/9-20-special-san-francisco> YouTube
<http://www.youtube.com/920special>
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
