-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change "NA" to "None" or 0 #658
Comments
I suggest to use zero value only when the value from backend is really reported as zero, and NA when the value is not available (so that there is nothing to report). @julienlim Would it be ok to use Which means that in this case, we need to figure out where the problem actually is:
In other words, this is not just a sheer change of the label, and we need to be sure that the backed works before trying to fix this. |
I think there are actually three conditions that should be differentiated.
|
@julienlim The Issue has been fixed, Please verify with the master build(tendrl-ui-1.5.2-20171011T095120.b554d56.src.rpm). |
For views where "NA" is used, NA is too vague. Please instead use the following:
Hosts List:
Hosts List, Volumes List:
@gnehapk @a2batic @Tendrl/tendrl-qe @mcarrano
The text was updated successfully, but these errors were encountered: