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

Add vlans data source #420

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Add vlans data source #420

merged 1 commit into from
Jun 27, 2023

Conversation

danischm
Copy link
Contributor

No description provided.

@fbreckle
Copy link
Collaborator

fbreckle commented Jun 27, 2023

Hi!

Thanks for your MR. LGTM. I took the liberty and removed an unwanted change in the docs that happened when you have a local NETBOX_API_TOKEN and/or NETBOX_SERVER_URL set.
I updated the README of the repo to reference the new make-target that circumvents this issue in de70636

@fbreckle fbreckle merged commit be29ccd into e-breuninger:master Jun 27, 2023
@danischm
Copy link
Contributor Author

Great, thx! If you have a chance to release a new version anytime soon, that would be great.

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.

2 participants