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

PM-411 - Service References Unique Constraints #428

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

jamesditrapani
Copy link
Contributor

Fixes:

Closes #411

Stop service_reference being generated as a UUID. Convert to a Unique ID based on the Database ID of a new ServiceReference model.

To fix:

Unique constraints on service_reference do not carry over between Direct & IX Peering Sessions.

@jamesditrapani jamesditrapani self-assigned this May 23, 2021
@jamesditrapani jamesditrapani added this to In progress in Core via automation May 23, 2021
extras/models.py Show resolved Hide resolved
extras/models.py Show resolved Hide resolved
extras/models.py Outdated Show resolved Hide resolved
@gmazoyer gmazoyer added the status: do not merge Do not merge the code specified for now label Jun 4, 2021
@jamesditrapani
Copy link
Contributor Author

Moved to backlog, will be addressed in the future.

@github-actions
Copy link

This PR has been automatically marked as stale because it has not had recent activity. It will be closed automatically if no further action is taken.

@jamesditrapani
Copy link
Contributor Author

Keeping open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted status: do not merge Do not merge the code specified for now
Projects
No open projects
Core
In progress
Development

Successfully merging this pull request may close these issues.

New service reference is not unique enough
2 participants