Skip to content
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

feat: include a alias_display attr for Switch, Link, and Interface and expose on their dict and json values #493

Open
viniarck opened this issue Aug 8, 2024 · 0 comments
Labels
enhancement New feature or request priority_low Low priority

Comments

@viniarck
Copy link
Member

viniarck commented Aug 8, 2024

In the front-end, there's always that requirement of trying to display user friendly displayed value for a Switch, Link, and Interface, which boils down to trying to display a metadata.[node_name|link_name|port_name] (respectively) or fallback to their id value if the metadata value is empty or not set. This code has been repeated all over many components in the UI, this should be implemented in the back-end and then the UI just read this alias_display or whatever it'll be called.

Here are a few examples where that gets (re)implemented over and over in many UIs components:

@viniarck viniarck added enhancement New feature or request priority_low Low priority labels Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority_low Low priority
Projects
None yet
Development

No branches or pull requests

1 participant