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

Added custom-field-choice-sets to nb_lookup.py #1186

Merged
merged 2 commits into from
Apr 7, 2024

Conversation

MrDiba
Copy link
Contributor

@MrDiba MrDiba commented Mar 7, 2024

Related Issue

#1185

New Behavior

Since NetBox v3.6.0 there is an breaking change where the custom-fields and choices are split.

Contrast to Current Behavior

Currently there is no way to access the custom field choices when using NetBox v3.6.0 or higher

Discussion: Benefits and Drawbacks

Why do you think this project and the community will benefit from your proposed change?

Starting from NetBox v3.6.0 it is not possible to use the lookup module to get the custom field choices.

What are the drawbacks of this change?

I do not know what happens when someone using it with an NetBox version lower than v3.6.0.

Is it backwards-compatible?

NetBox <v3.6.0; Use custom-fields to get the choices.
Netbox >v3.6.0; Use custom-field-choice-sets

Changes to the Documentation

...

Proposed Release Note Entry

Added custom-field-choice-sets to the lookup plugin.

Double Check

  • I have read the comments and followed the CONTRIBUTING.md.
  • I have explained my PR according to the information in the comments or in a linked issue.
  • My PR targets the devel branch.

Since NetBox v3.6.0 the custom fields and the choices are splitted.
@MrDiba MrDiba requested a review from mariolenz April 3, 2024 09:44
sc68cal
sc68cal previously requested changes Apr 7, 2024
Copy link
Contributor

@sc68cal sc68cal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to make this change in a way that it supports NetBox versions less than 3.6 since we still support those per our policy

@sc68cal sc68cal dismissed their stale review April 7, 2024 16:58

mistake

@sc68cal sc68cal merged commit a24f226 into netbox-community:devel Apr 7, 2024
8 checks passed
@MrDiba MrDiba deleted the patch-2 branch April 8, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants