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.

Sign up for GitHub

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

Jump to bottom

Regression from rack group filtering fix #3895

Closed
hSaria opened this issue Jan 11, 2020 · 0 comments · Fixed by #3896
Closed

Regression from rack group filtering fix #3895

hSaria opened this issue Jan 11, 2020 · 0 comments · Fixed by #3896
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application

Comments

@hSaria
Copy link
Contributor

hSaria commented Jan 11, 2020 •
edited
Loading

Environment

  • Python version: 3.6.8
  • NetBox version: 2.6.12-dev

Steps to Reproduce

  1. Navigate to /dcim/rack-elevations/

Expected Behavior

Page loads

Observed Behavior

Exception

<class 'KeyError'>

'rack_group_id'

This is a regression from 6bc8f2e#diff-20da25434db0391c8eafd320dff942b3L742-R742.
@hSaria hSaria mentioned this issue Jan 11, 2020
Merged
@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application labels Jan 13, 2020
jeremystretch added a commit that referenced this issue Jan 13, 2020
@jeremystretch
Merge pull request #3896 from hSaria/3895-elevations-keyerror
2f98f13 
Fixes #3895: Elevations filter regression
@lock lock bot locked as resolved and limited conversation to collaborators Apr 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #3895: Elevations filter regression
2 participants
@jeremystretch @hSaria