-
Notifications
You must be signed in to change notification settings - Fork 19
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 documentation #4
Conversation
README.md
Outdated
|
||
``` | ||
pip install netbox-virtual-circuit-plugin |
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.
is this project hosted on pypi?
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.
Yes that’s the plan
README.md
Outdated
Virtual Circuit, it can not exist in another Virtual Circuit without first being | ||
removed. |
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 not sure I understand what this sentence is saying
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.
lgtm!
my only question here is whether we will want this published on pypi since it seems like something we'd only use internally, but I may not be thinking of all use cases, etc. so that can be a discussion for later
Good point. I will open a separate issue to keep track of this. As for now, my only thought for publishing it to pypi is that it would save us a ton of time to install and enable the plugin in Netbox, though I might not know if there's also another good way to do so. |
No description provided.