markobean commented on PR #10097:
URL: https://github.com/apache/nifi/pull/10097#issuecomment-3168321846

   > Reviewing the latest updates, this looks close to completion.
   > 
   > I noticed one minor issue that could lead to confusion related to 
filtering. `Active` and `Inactive` work, but filtering on `Not Configured` does 
not work because it doesn't match the enumerated value, but `NOT_CONFIGURED` 
does work. This is not the most intuitive behavior, given the displayed value 
is `Not Configured`. One option is to render `NOT_CONFIGURED` for the column 
value. Another option could be to replace the space character in the 
`filterTerm` with `_` to allow matching.
   
   Seems reasonable. I modified the filtering for the loadBalanceStatus column 
so that it replaces spaces with underscores. Now, it will match the enum value. 
For good measure, I also trimmed the search term to eliminate leading and 
trailing whitespace that may inadvertently add erroneous underscores.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to