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

Change "Interface" to "Tunnel interface" in tunnel termination form #14707

Closed
markkuleinio opened this issue Jan 5, 2024 · 1 comment · Fixed by #15624
Closed

Change "Interface" to "Tunnel interface" in tunnel termination form #14707

markkuleinio opened this issue Jan 5, 2024 · 1 comment · Fixed by #15624
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application

Comments

@markkuleinio
Copy link
Contributor

markkuleinio commented Jan 5, 2024

NetBox version

v3.7.0

Feature type

Change to existing functionality

Proposed functionality

I'm proposing changing the "Interface" field to "Tunnel interface" in the VPN tunnel termination form.

Use case

The current terminology talking about "tunnel termination" and then asking for interface hints the user that this is the actual tunnel starting point (outside interface).

However, the real purpose of the field is to designate the route-based tunnel interface (inside interface), not the tunnel source/destination (outside) interface. This is somewhat confusing, especially when the next field is called "Outside IP" that definitely talks about the tunnel outside interface. (That's logical thinking in other cases for me at least: first you select interface, then you select the IP of that interface)

Example issue of the current confusion is #14681.

Renaming the interface field to "Tunnel interface" would guide the user to use the tunnel (inside) interface, not the outside interface.

Database changes

Not sure if this naming change should involve changing some internal structures as well

External dependencies

No response

@markkuleinio markkuleinio added the type: feature Introduction of new functionality to the application label Jan 5, 2024
@jeremystretch jeremystretch added the status: needs triage This issue is awaiting triage by a maintainer label Mar 26, 2024
@jeffgdotorg jeffgdotorg added status: accepted This issue has been accepted for implementation and removed status: needs triage This issue is awaiting triage by a maintainer labels Apr 3, 2024
@jeffgdotorg
Copy link
Contributor

If I've understood your proposal correctly, this change can be made without model changes, just a wording change to the tunnel termination creation form. Go ahead with a PR if you agree.

markkuleinio added a commit to markkuleinio/netbox that referenced this issue Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: feature Introduction of new functionality to the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants