[feature] REST API: allow filtering devices by location ID and floorplan ID

[nemesifier]

Add the following filters to the device list API endpoint:

• location: filter devices having a DeviceLocation object relating to the location ID
• floorplan: filter devices having a DeviceLocation object relating to the floorplan ID

[Piyushrathoree]

@nemesifier can i work on this ?

[lakshay909]

Hi @nemesifier , I’d like to work on this issue — thank you for maintaining such a well-structured project!

I’m a student developer experienced in Python, Django, and REST API integrations, with prior contributions to open-source projects focused on backend optimization and code quality improvements. My approach will involve reproducing the issue locally, reviewing the related module logic, and implementing a consistent fix following OpenWISP’s coding standards and test coverage requirements. I’ll also ensure thorough testing and documentation updates as needed.

I can submit a PR within 3–5 days once assigned. Kindly assign this issue to me — I’d be glad to contribute!

[pandafy]

Hey @lakshay909, I think @Piyushrathoree is already working on this.

@Piyushrathoree can you confirm if we are already working on this issue?

[Piyushrathoree]

Hey @lakshay909, I think @Piyushrathoree is already working on this.

@Piyushrathoree can you confirm if we are already working on this issue?

Yeah I'm working on this , and soon raise the PR too

[ssmadhavan006]

Hi, I saw that this task isn't closed as of now. I would like to work on this

[Srinath0916]

Hi @nemesifier, If this issue is still unassigned / pending, I’d be happy to take it up. I recently submitted a PR for another issue in this repo, so I can start working on this one immediately as well Please assign the issue to me when convenient. I’ll share the PR when it’s ready.

[ssmadhavan006]

Hi, @Srinath0916, I am currently working on this

[Srinath0916]

Thanks for the update @ssmadhavan006.

Just to clarify, I actually started working on this a 2 days ago and the implementation along with the required test cases is already complete on my local setup.

Since there is currently no PR open for this issue, I’ll go ahead and submit one so the maintainers can review it, if u hadn't completed the issue yet.

[ssmadhavan006]

Hey @Srinath0916 , thanks for letting me know and updating the thread. I appreciate your initiative on this! I also started working on this a couple of days ago and have been making progress locally, but since you already have a finished implementation, please go ahead and submit the PR for review. If there’s any feedback, additional improvements, or testing needed after your PR is up, I’d be happy to help with review or further enhancements! Looking forward to seeing your solution merged.