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

Device name filtering with multiple parameters not working #2577

Closed
kirpit opened this issue Nov 9, 2018 · 0 comments · Fixed by #2992
Closed

Device name filtering with multiple parameters not working #2577

kirpit opened this issue Nov 9, 2018 · 0 comments · Fixed by #2992
Labels
status: accepted This issue has been accepted for implementation type: documentation A change or addition to the documentation

Comments

@kirpit
Copy link

kirpit commented Nov 9, 2018

Environment

  • Python version: 3.6.3
  • NetBox version: Latest dev (61ca7ee)

Steps to Reproduce

As it is described in the docs, we can apply same filter with multiple values:

GET /api/ipam/prefixes/?status=1&status=2

Unfortunately, this doesn't work for specifically the "name" field for devices endpoint and it returns only 1 result of the last queried value.

GET /api/dcim/devices/?name=foo&name=bar&name=baz

Expected Behavior

Endpoint to return all 3 devices; foo, bar, baz.

Observed Behavior

Endpoint returning only 1 device; baz.

========
I am happy to help, if pointed in the right direction for the fix and to have a test.

@DanSheps DanSheps added type: documentation A change or addition to the documentation status: accepted This issue has been accepted for implementation labels Mar 11, 2019
DanSheps added a commit that referenced this issue Mar 11, 2019
* Fixes #2991: Fixed typo
* Fixes #2577: Clarified documentation regarding filtering multiple times (some filters can, some cannot)
@lock lock bot locked as resolved and limited conversation to collaborators Jan 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: documentation A change or addition to the documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants