-
Notifications
You must be signed in to change notification settings - Fork 264
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 "Data Model Changes" section to "Migrating from NetBox" #121
Add "Data Model Changes" section to "Migrating from NetBox" #121
Conversation
- Explaining difference in importing objects with a `status` field
|
||
## Data Model Changes | ||
|
||
The following backwards-incompatible changes have been made to the data model. |
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.
What's our plan to move forward with this section? Will this become an ever-growing list of changes as we diverge further from NetBox? Are we adding content here that's better suited to the Release Notes and/or the per-model documentation?
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 am thinking it would be good to have them tracked here centrally, as the release notes start to fade into history.
We might need to decide for how long we intend to maintain this guide. As I see it, it's a "for now", and not a "forever".
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.
Want to add a note about the primary key int/UUID change as well?
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.
You know what. I was going to do that but decided to leave it for another PR.
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, thanks a lot @jathanism
|
||
## Data Model Changes | ||
|
||
The following backwards-incompatible changes have been made to the data model. |
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.
Want to add a note about the primary key int/UUID change as well?
@glennmatthews I went ahead and added a note about UUID fields. 📦 |
status
fieldRelated to: #62