wasn’t obvious to me since i actually changed the code of earnings.py before i figured it out… and sure in hindsight it makes perfect sense like i suggested above…
but that does assume prior knowledge.
i don’t understand what you mean by this, i mean i understand… but what are you talking about the status OK
because the problem was the the warnings removed the OK as shown in the screenshots above.
i don’t agree that it’s clearly obvious, without prior knowledge when it looks like this.
i wasn’t expecting my node to be done vetting yet on any satellites, maybe thats what got me and i will agree it’s very much an edge case… in many cases it would be fine because one will be able to see, unvetted nodes with errors.
so i guess this particular case is very rare… but still
i will however say, i can see how it’s difficult to make a better solution… no matter how i want to fix it i find issues with how its written…
can’t add the % in front of the warning because thats confusing. can’t/ don’t want to keep the % vetted because it takes up space.
think i would just do a minimal change and add the 100% vetted at the end of the warnings like it does when it’s not vetted… seems to be room enough for it and don’t think it will look to confusing.
and shouldn’t in any way be able to be confused with anything else, and it would be minimal code change basically just copy pasting already existing code.