Skip to content

10458 add "Location" to CircuitTermination #10458

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

Conversation

gjvc
Copy link

@gjvc gjvc commented Sep 26, 2022

Fixes: #9604

added location field to ProviderForm
added location foreign key to CircuitTermination
added location field to circuittermination_edit.html template
updated circuits table to display Side A/B value via this logic:

  • if location is set, show location
  • if site only is set, show site (as before)
  • if provider_network only is set, show provider_network (as before)

gjvc added 7 commits June 25, 2022 21:48
if location is set, show location
if site only is set, show site (as before)
if provider_network only is set, show provider_network (as before)
if location is set, show location
if site only is set, show site (as before)
if provider_network only is set, show provider_network (as before)
@gjvc gjvc changed the title add "Location" to CircuitTermination 10458 add "Location" to CircuitTermination Sep 26, 2022
@gjvc gjvc marked this pull request as ready for review September 26, 2022 07:53
@jeremystretch
Copy link
Member

Thanks for your interest in working on this, however please note that #9604 has not been assigned a release milestone (required as this would be a breaking change to the API) and thus is not yet ready for work.

@gjvc
Copy link
Author

gjvc commented Sep 26, 2022

Sure thing; I will keep an eye on the original issue and this PR and will resubmit when you are ready. Thank you for your consideration.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Location to Circuit Termination
2 participants