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

V2/2.0.0b2 compatibility #234

Merged
merged 53 commits into from
Oct 2, 2023
Merged

V2/2.0.0b2 compatibility #234

merged 53 commits into from
Oct 2, 2023

Conversation

pszulczewski
Copy link
Contributor

@pszulczewski pszulczewski commented Aug 17, 2023

Updates for Nautobot 2.X

IMPLEMENTATION DECISIONS:

  • Locations are implemented in a way, that referencing a location by other models can be only done by passing location id (PK). Similarly in location module, location can only be deleted by providing location id.
    This decision was made based due the fact that locations are hierarchical and name doesn't guarantee that a unique location will be found. This is left to the user to make a unique query (lookup module) to find the location. This aligns to the core team recommendation that for locations PK or a unique set of parameters should be used to identify a location.

Bumps [certifi](https://github.com/certifi/python-certifi) from 2022.12.7 to 2023.7.22.
- [Commits](certifi/python-certifi@2022.12.07...2023.07.22)

---
updated-dependencies:
- dependency-name: certifi
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
@pszulczewski pszulczewski added this to the Nautobot v2 milestone Aug 17, 2023
@@ -0,0 +1 @@
nautobot-bgp-models @ git+https://github.com/nautobot/nautobot-plugin-bgp-models@v2
Copy link
Contributor

Choose a reason for hiding this comment

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

I ripped out these conditional requirement files in #242. Do we plan on keeping separate 1.X/2.X branches? If so, we should just change the standard requirements file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, there will be separation between 1.X and 2.X.
I have seen this PR you mentioned above, I'll have to re-base and follow that.

@pszulczewski pszulczewski force-pushed the v2/2.0.0b2_compatibility branch from 4c3834b to 9f2a151 Compare September 11, 2023 11:29
@pszulczewski pszulczewski force-pushed the v2/2.0.0b2_compatibility branch from 9f2a151 to 284914f Compare September 11, 2023 11:38
@pszulczewski pszulczewski force-pushed the v2/2.0.0b2_compatibility branch from c3c49e6 to d7847aa Compare September 21, 2023 18:47
@pszulczewski pszulczewski force-pushed the v2/2.0.0b2_compatibility branch from d7847aa to 549c7db Compare September 21, 2023 18:50
@pszulczewski
Copy link
Contributor Author

Plugin module has issues with getting status field converted to ID. I will look into that while working on issue #244 as it seems to be related. For now marked as TODO.

@pszulczewski pszulczewski marked this pull request as ready for review September 30, 2023 13:06
@pszulczewski pszulczewski merged commit 66c6bc8 into next Oct 2, 2023
@pszulczewski pszulczewski deleted the v2/2.0.0b2_compatibility branch October 11, 2023 13:56
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.

4 participants