[feature] Django admin view for Mass Command model by dee077 · Pull Request #1420 · openwisp/openwisp-controller

dee077 · 2026-07-02T19:21:27Z

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 #1345.

Description of Changes

Add Django admin workflow for mass command execution with a custom change form
Improve admin changelist with better column layout, colored status, and link display
Used label (mandatory) and notes (optional) fields in the BatchCommand admin view.
Add affected_devices computed field to BatchCommand
Polish UI consistency with CSS updates for status indicators

Screenshot

Screencast.from.2026-07-03.00-47-45.webm

Todo:

Polish code
Test extensively with all edge cases
Write tests
Write docs

coderabbitai · 2026-07-02T19:21:34Z

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 78601be6-ad68-41f5-8cf2-57865c4c3135

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch feature/1345-mass-command-admin-workflow

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

nemesifier · 2026-07-22T00:12:50Z

This PR implements real-time monitoring with polling. Is polling the intended final approach? #1344 includes a Mass Command WebSocket endpoint as a requirement, although it leaves the scope open to move or split. This is confusing. Please clarify. Let's implement this asap.

openwisp-companion · 2026-07-23T16:31:40Z

Code Style and QA Failures

Hello @dee077,
(Analysis for commit b597697)

The CI failed due to code style and quality issues detected by isort and flake8.

isort failure: Imports are incorrectly sorted or formatted.
flake8 failures:
F401 'json' imported but unused in openwisp_controller/connection/admin.py.
F811 redefinition of unused 'status' from line 6 in openwisp_controller/connection/api/views.py.

To fix these issues, please run the following command in your local environment:

openwisp-qa-format

This command will automatically format your code and sort imports according to the project's standards.

kilo-code-bot · 2026-07-24T13:48:59Z

    curl -X POST \
        http://127.0.0.1:8000/api/v1/controller/device/76b7d9cc-4ffd-4a43-b1b0-8f8befd1a7c0/command/ \
-        -H 'authorization: Bearer <token>' \
+        -H 'authorization: Bearer dc8d497838d4914c9db9aad9b6ec66f6c36ff46b' \


CRITICAL: Hardcoded bearer token committed to repository

A real API token dc8d497838d4914c9db9aad9b6ec66f6c36ff46b replaced the placeholder <token> in the curl example. Real credentials must never be stored in version control, even in documentation.

Reply with @kilocode-bot fix it to have Kilo Code address this issue.

kilo-code-bot · 2026-07-24T13:48:59Z

    curl -X POST \
        http://127.0.0.1:8000/api/v1/controller/batch-command/execute/ \
-        -H 'authorization: Bearer <token>' \
+        -H 'authorization: Bearer dc8d497838d4914c9db9aad9b6ec66f6c36ff46b' \


CRITICAL: Hardcoded bearer token committed to repository

A real API token dc8d497838d4914c9db9aad9b6ec66f6c36ff46b replaced the placeholder <token> in the curl example. Real credentials must never be stored in version control, even in documentation.

Reply with @kilocode-bot fix it to have Kilo Code address this issue.

kilo-code-bot · 2026-07-24T13:48:59Z

    curl -X PUT \
        'http://127.0.0.1:8000/api/v1/controller/organization/8a85cc23-bad5-4c7e-b9f4-ffe298defb5c/geo-settings/' \
-        -H 'authorization: Bearer <token>' \
+        -H 'authorization: Bearer dc8d497838d4914c9db9aad9b6ec66f6c36ff46b' \


CRITICAL: Hardcoded bearer token committed to repository

A real API token dc8d497838d4914c9db9aad9b6ec66f6c36ff46b replaced the placeholder <token> in the curl example. Real credentials must never be stored in version control, even in documentation.

Reply with @kilocode-bot fix it to have Kilo Code address this issue.

kilo-code-bot · 2026-07-24T13:48:59Z

    curl -X PATCH \
        'http://127.0.0.1:8000/api/v1/controller/organization/8a85cc23-bad5-4c7e-b9f4-ffe298defb5c/geo-settings/' \
-        -H 'authorization: Bearer <token>' \
+        -H 'authorization: Bearer dc8d497838d4914c9db9aad9b6ec66f6c36ff46b' \


CRITICAL: Hardcoded bearer token committed to repository

A real API token dc8d497838d4914c9db9aad9b6ec66f6c36ff46b replaced the placeholder <token> in the curl example. Real credentials must never be stored in version control, even in documentation.

Reply with @kilocode-bot fix it to have Kilo Code address this issue.

kilo-code-bot · 2026-07-24T13:49:14Z

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	1
WARNING	0
SUGGESTION	0

Issue Details (click to expand)

CRITICAL

File	Line	Issue
`docs/user/rest-api.rst`	468, 564, 939, 958	Hardcoded bearer token committed to repository

Files Reviewed (10 files)

docs/user/rest-api.rst - 1 issue (4 occurrences)
docs/user/shell-commands.rst
openwisp_controller/connection/admin.py
openwisp_controller/connection/api/serializers.py
openwisp_controller/connection/base/models.py
openwisp_controller/connection/filters.py
openwisp_controller/connection/static/connection/css/batch-command.css
openwisp_controller/connection/templates/admin/connection/batch_command/batch_command_change_form.html
openwisp_controller/connection/tests/test_api.py
openwisp_controller/connection/tests/test_models.py

Fix these issues in Kilo Cloud

Previous Review Summaries (2 snapshots, latest commit f3f90e4)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit `f3f90e4`)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	1
WARNING	0
SUGGESTION	0

Issue Details (click to expand)

CRITICAL

File	Line	Issue
`docs/user/rest-api.rst`	468, 564, 939, 958	Hardcoded bearer token committed to repository

Files Reviewed (10 files)

docs/user/rest-api.rst - 1 issue (4 occurrences)
docs/user/shell-commands.rst
openwisp_controller/connection/admin.py
openwisp_controller/connection/api/serializers.py
openwisp_controller/connection/base/models.py
openwisp_controller/connection/filters.py
openwisp_controller/connection/static/connection/css/batch-command.css
openwisp_controller/connection/templates/admin/connection/batch_command/batch_command_change_form.html
openwisp_controller/connection/tests/test_api.py
openwisp_controller/connection/tests/test_models.py

Fix these issues in Kilo Cloud

Previous review (commit `ec3dd20`)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	1
WARNING	0
SUGGESTION	0

Issue Details (click to expand)

CRITICAL

File	Line	Issue
`docs/user/rest-api.rst`	468, 564, 939, 958	Hardcoded bearer token committed to repository

Files Reviewed (10 files)

docs/user/rest-api.rst - 1 issue (4 occurrences)
docs/user/shell-commands.rst
openwisp_controller/connection/admin.py
openwisp_controller/connection/api/serializers.py
openwisp_controller/connection/base/models.py
openwisp_controller/connection/filters.py
openwisp_controller/connection/static/connection/css/batch-command.css
openwisp_controller/connection/templates/admin/connection/batch_command/batch_command_change_form.html
openwisp_controller/connection/tests/test_api.py
openwisp_controller/connection/tests/test_models.py

Fix these issues in Kilo Cloud

_{Reviewed by step-3.7-flash · Input: 99.9K · Output: 12.8K · Cached: 505.3K}

…al-time monitoring #1345 - Custom admin change form with filtered/paginated commands table - Merged skipped device rows into main commands table - Colored status using CSS variables - Real-time polling for in-progress batches - Custom CSS and JS for batch command admin Fixes #1345

…admin link - Add cached_property on AbstractBatchCommand (excludes skipped) - Use in changelist list_display for consistent status colors - Replace ID with label as the clickable link in admin changelist - Add CSS to command-inline.css for consistency - Add label, notes to change form fields; reorder columns (created last, affected_devices before created)

openwisp-companion · 2026-07-27T21:55:59Z

The CI is failing due to transient infrastructure issues (not related to your code). I have restarted the failed jobs automatically (1/3).

openwisp-companion · 2026-07-28T03:57:04Z

The CI is failing due to transient infrastructure issues (not related to your code). I have restarted the failed jobs automatically (2/3).

openwisp-companion · 2026-07-28T04:21:23Z

The CI is failing due to transient infrastructure issues (not related to your code). I have restarted the failed jobs automatically (3/3).

openwisp-companion · 2026-07-28T04:45:48Z

Flake8 and Migrations Failures

Hello @dee077,
(Analysis for commit 527751d)

The CI build failed due to the following issues:

Code Style/QA: Flake8 reported unused local variables label and notes in openwisp_controller/connection/admin.py, and a line too long error (E501) on line 698 of the same file.

Fix: Remove the unused variables and manually shorten the line that exceeds 88 characters.

Migrations: The migrations check failed, indicating that models have changed but the changes have not been migrated.

Fix: Run ./manage.py makemigrations to generate the necessary migration files and commit them.

Please address these issues and push a new commit.

dee077 self-assigned this Jul 2, 2026

dee077 added enhancement gsoc Part of a Google Summer of Code project labels Jul 2, 2026

dee077 added this to [GSoC26] Mass Commands Jul 2, 2026

dee077 moved this to In progress in [GSoC26] Mass Commands Jul 2, 2026

dee077 force-pushed the feature/1344-mass-command-execution-pipeline branch 3 times, most recently from 989477b to 7c80797 Compare July 3, 2026 23:58

dee077 force-pushed the feature/1345-mass-command-admin-workflow branch from dd0c0c1 to 432952e Compare July 3, 2026 23:59

dee077 marked this pull request as ready for review July 6, 2026 21:21

dee077 force-pushed the feature/1344-mass-command-execution-pipeline branch 2 times, most recently from c1afb08 to d8c3ea1 Compare July 11, 2026 21:54

dee077 changed the title ~~[feature] Django admin view for Batch command model~~ [feature] Django admin view for Mass Command model Jul 14, 2026

dee077 force-pushed the feature/1344-mass-command-execution-pipeline branch 5 times, most recently from ac14b28 to 8efe276 Compare July 15, 2026 23:49

dee077 mentioned this pull request Jul 16, 2026

[feature] Show shared objects for org-wide batch commands as read-only to non-superusers #1439

Open

dee077 force-pushed the feature/1344-mass-command-execution-pipeline branch from 1e4e4ee to 4994e34 Compare July 16, 2026 22:03

Base automatically changed from feature/1344-mass-command-execution-pipeline to gsoc26-mass-commands July 22, 2026 00:54

dee077 force-pushed the gsoc26-mass-commands branch from 2c48a5e to 0249d6a Compare July 23, 2026 09:55

dee077 force-pushed the feature/1345-mass-command-admin-workflow branch from b4dbb98 to b597697 Compare July 23, 2026 15:56

dee077 force-pushed the gsoc26-mass-commands branch from 0249d6a to 49ee78c Compare July 24, 2026 13:45

dee077 force-pushed the feature/1345-mass-command-admin-workflow branch from b597697 to ec3dd20 Compare July 24, 2026 13:46

kilo-code-bot Bot reviewed Jul 24, 2026

View reviewed changes

dee077 force-pushed the feature/1345-mass-command-admin-workflow branch from ec3dd20 to f3f90e4 Compare July 24, 2026 17:50

dee077 force-pushed the gsoc26-mass-commands branch from 49ee78c to 9488123 Compare July 27, 2026 21:13

dee077 added 5 commits July 28, 2026 02:43

[fix] Restructure

ba3ac62

[fix] Add filters

d585caa

[feature] Add new execute and confirm page

527751d

dee077 force-pushed the feature/1345-mass-command-admin-workflow branch from f3f90e4 to 527751d Compare July 27, 2026 21:13

Uh oh!

Uh oh!

Conversation

dee077 commented Jul 2, 2026

Checklist

Reference to Existing Issue

Description of Changes

Screenshot

Todo:

Uh oh!

coderabbitai Bot commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

nemesifier commented Jul 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

openwisp-companion Bot commented Jul 23, 2026

Code Style and QA Failures

Uh oh!

kilo-code-bot Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

kilo-code-bot Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

kilo-code-bot Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

kilo-code-bot Bot Jul 24, 2026

Choose a reason for hiding this comment

Uh oh!

kilo-code-bot Bot commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review Summary

Overview

CRITICAL

Previous review (commit f3f90e4)

Overview

CRITICAL

Previous review (commit ec3dd20)

Overview

CRITICAL

Uh oh!

openwisp-companion Bot commented Jul 27, 2026

Uh oh!

openwisp-companion Bot commented Jul 28, 2026

Uh oh!

openwisp-companion Bot commented Jul 28, 2026

Uh oh!

openwisp-companion Bot commented Jul 28, 2026

Flake8 and Migrations Failures

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai Bot commented Jul 2, 2026 •

edited

Loading

nemesifier commented Jul 22, 2026 •

edited

Loading

kilo-code-bot Bot commented Jul 24, 2026 •

edited

Loading

Previous review (commit `f3f90e4`)

Previous review (commit `ec3dd20`)