Github user mcgilman commented on the issue:

    https://github.com/apache/nifi/pull/583
  
    Hey @jvwing... I had a chance to check out the update. Everything looks 
good but I do have one other comment that I probably should have noticed 
before. Given the build details ultimately come from `nifi.properties` I don't 
think we can rely on their presence. Often times, folks upgrade by retaining 
their consisting `conf` directory so we typically try to ensure things still 
work when the properties do not exist. In the System Diagnostics, it probably 
makes sense to simply populate the fields with a message indicating that the 
value isn't set. Additionally, you can style the elements with `unset`. In the 
About dialog is probably makes sense to just not render anything under the 
version. Currently, it just says 'Tagged'.
    
    Thoughts?



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to