Feature #14793
closed
- Status changed from New to Rejected
This is not needed. SFP information is already printed on Status > Interfaces. If more detail is needed the additional fields could be added there rather than making a whole separate package for it.
I have a totally different opinion on this, the existing interface output is cluttered and not showing all needed information. We run dozed of pfSense systems remotely where this view would be a nice addon. I didnt make a PR to add this to the main functionality it is a Package so people can decide if they want to use it or not. Just a reject without any further discussion on this... common it is a package not a core function.
And just double checked Jim Pingle in scenarios like having a LAG the Mentioned Interface Status is not displaying any SFP information at all.
So as said, it is a optional package, dont see why this should be rejected completely.
Jim Pingle wrote in #note-1:
This is not needed. SFP information is already printed on Status > Interfaces. If more detail is needed the additional fields could be added there rather than making a whole separate package for it.
Jim Pingle please do not just ignore this. Reject in minutes but not even answering valid points, this is not how open source should work.
If status_interfaces.php is insufficient in some way, the correct thing to do would be to fix or otherwise improve that page, proposing changes and getting approval before any development work. Having an entire extra package adds technical debt and unnecessarily complicates the process.
This reason is valid and true. I will think about starting a suggestion on how to improve the interface list.
Thank you
Also available in: Atom
PDF