-
Notifications
You must be signed in to change notification settings - Fork 15
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
Fixes: Netbox 4.0 breaking change #123
Conversation
…ce.role. This fixes adds logic to handle this change across Netbox versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm running Netbox 4.0.5 and with this changes in nxr.py worked for me. Thanks!
@bortok Have you got a moment to review this one? |
@tardoe sorry for the delay, I'm traveling this month. The reason I didn't merge it right away is the need to bump netbox versions in the CI environment to test with 4.0. |
Hi @tardoe, I'm working on this. Could you please cancel this PR and make a new one with |
it has to be under the role setting if block to make sure the role is defined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested with NB v4.0
Renames device.device_role to device.role. This fixes adds logic to handle this change across Netbox versions.