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

Changing org while adding net results in 500 #654

Closed
arnoldnipper opened this issue Mar 2, 2020 · 6 comments · Fixed by #751
Closed

Changing org while adding net results in 500 #654

arnoldnipper opened this issue Mar 2, 2020 · 6 comments · Fixed by #751
Assignees
Labels
bug Time:Minor Up to 4 hours
Milestone

Comments

@arnoldnipper
Copy link
Contributor

Scenario:

  • user sends a request for adding a net under his org
  • According to RIR information the org is not the owner
  • AC adds new org via looking glass and "+"
  • New org shows up in net object under approval
  • Clicking on "Save" results in "Server Error (500)"

Workaround is to set the new org_id via API

@peeringdb/ac can you confirm?

@arnoldnipper arnoldnipper self-assigned this Mar 2, 2020
@arnoldnipper arnoldnipper added this to the 1 Decide milestone Mar 2, 2020
@netravnen
Copy link
Contributor

@peeringdb/ac can you confirm?

Confirmed. (Tested in Chrome, Chrome Incognito, Internet Explore, and Chrome Mobile (Android))

@arnoldnipper
Copy link
Contributor Author

Looks it has to do with unset prefix fields

@mcmanuss8
Copy link
Contributor

That's a bummer. Let's fix this bug. +1

@grizz
Copy link
Member

grizz commented Apr 18, 2020

+1

@koalafil
Copy link

koalafil commented Jun 1, 2020

Can we have the Release Notes pls?

@arnoldnipper
Copy link
Contributor Author

Release Note

Bugfix for the Admin Committee UI.

vegu pushed a commit that referenced this issue Jun 24, 2020
Changing org while adding net results in 500 #654

See merge request gh/peeringdb/peeringdb!77
@grizz grizz mentioned this issue Jun 24, 2020
grizz added a commit that referenced this issue Jun 24, 2020
* Add pointer from API docs to tutorial #650

* Sorting by clicking table headers should use local-compare #356

* Mark IXP peering LAN as bogon #352

* Add help text to "Add (Facility, Network, Exchange)" tab #669

* Add Looking Glass field to the IX object #672

* Add read-only Superuser #679

* Make spelling of traffic levels consistent #519 (#723)

* Offer 2FA (#290)

* Show "Last Updated" fields on fac, ix, org records (#526)

* Enable sort and reverse sort of IP column in IX display (#72)

* IRR validation not handling unexpected characters gracefully (#712)

* Support alternative direction of writing, e.g. Arabic (#618)

* Undeleting an ixlan with an emtpy IPv4 XOR IPv6 field throws a silly error (#644)

* Changing org while adding net results in 500 #654

* missing delete button for organisations (#121)

* When changing owner of an ix admin GUI borks because of "Ixlan for exchange already exists" #666

* Selection should only present undeleted objects (#664)

* change default encoding of API calls to 'utf-8' #663

* Posting https://www.peeringdb.com onto social media doesn't select a good preview image #537

* Revert "Add Looking Glass field to the IX object #672"

This reverts commit 4daf252.

Conflicts:
	peeringdb_server/migrations/0037_ix_looking_glass.py
	peeringdb_server/views.py

* 500 Internal Error when creating IX where prefix already exists elsewhere #718

* Fix graceful restore of soft-deleted objects with translation active (#580)

* Don't return any POC data with status=deleted #569
Hard delete soft-deleted pocs after grace period #566

* django-peeringdb from github@2.0.0.2-beta

Co-authored-by: Stefan Pratter <stefan@20c.com>
@vegu vegu modified the milestones: Next Release, 2.21 Jul 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Time:Minor Up to 4 hours
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants