Skip to content

[fix] Fixed device group filter label in device list #1071 - #1129

Merged
nemesifier merged 3 commits into
openwisp:masterfrom
panagiotisevaggelou:issues/1071-device-group-filter-label
Sep 30, 2025
Merged

[fix] Fixed device group filter label in device list #1071#1129
nemesifier merged 3 commits into
openwisp:masterfrom
panagiotisevaggelou:issues/1071-device-group-filter-label

Conversation

@panagiotisevaggelou

@panagiotisevaggelou panagiotisevaggelou commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

Closes #1071

Description of Changes

This PR fixes the label shown in the device group filter on the device list view.

Previously, when selecting the “no group” option, the dropdown displayed “Shared systemwide (no organization)”, which was confusing in this context.

Now, the filter shows a concise “-” instead, making it clearer and more user-friendly.

Other filters and widgets still display the original label unchanged.

(A copy of widget_attrs was created before updating it to ensure that only this filter’s widget behavior is modified, without unintentionally affecting other filters that also inherit from MultitenantRelatedOrgFilter.)

Screenshot

device-group-filter-label-fix

Replaced "Shared systemwide (no organization)" with "-" in the
device group filter dropdown of the device list view, while keeping
the original label unchanged elsewhere.

Fixes openwisp#1071
@coveralls

coveralls commented Sep 25, 2025

Copy link
Copy Markdown

Coverage Status

coverage: 98.706%. remained the same
when pulling 1ff1cdf on panagiotisevaggelou:issues/1071-device-group-filter-label
into 309f083 on openwisp:master.

@nemesifier nemesifier left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Comment thread openwisp_controller/config/filters.py Outdated
@github-project-automation github-project-automation Bot moved this from In progress to Reviewer approved in OpenWISP Priorities for next releases Sep 30, 2025
@nemesifier nemesifier changed the title [admin] Fixed device group filter label in device list #1071 [fix] Fixed device group filter label in device list #1071 Sep 30, 2025
@nemesifier
nemesifier merged commit ac6fd83 into openwisp:master Sep 30, 2025
13 checks passed
@github-project-automation github-project-automation Bot moved this from Reviewer approved to Done in OpenWISP Priorities for next releases Sep 30, 2025
@github-project-automation github-project-automation Bot moved this from In progress to Done in 25.09 Release Sep 30, 2025
@github-project-automation github-project-automation Bot moved this from In progress to Done in OpenWISP Contributor's Board Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project
Status: Done

Development

Successfully merging this pull request may close these issues.

[bug] Device group filter in admin shows "shared systemwide" instead of "no group"

3 participants