-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
remove add button if not needed. #12848
Comments
@sudheesh001 did you want to work on this one? |
Yes please, thank you so much! I'll take a shot at this as a part of the hackathon |
I'm sorry how does this FR make sense? The button was just recently added? Edit: Here's where they were added, why would we accept a FR to remove them again? Doesn't seem to make sense. I also see no reason why this was closed: #12847 @abhi1693 - why did you close that? It seems inconsistent with the rest of the views and as such valid from my perspective. |
@kkthxbye-code yes, look like these are reversed, this should be closed and #12847 should be opened / fixed. |
Closing this as the real issue should be fixed in #12847 |
NetBox version
v3.5.3
Feature type
Change to existing functionality
Proposed functionality
Since the add button is not suitable to be on the virtualization -> interfaces page as mentioned by @abhi1693 in #12847 , I propose to remove the add button in the image below.
Stay like on other pages that don't need the add button.
Use case
simplify usability.
Database changes
No response
External dependencies
No response
The text was updated successfully, but these errors were encountered: