django-loci icon indicating copy to clipboard operation
django-loci copied to clipboard

[fix] Address field real-time update #169

Open dee077 opened this issue 6 months ago • 4 comments

Added address update handling in the websocket message and updated listenForLocationUpdates logic accordingly.

Fixes #169

Checklist

  • [x] I have read the OpenWISP Contributing Guidelines.
  • [x] 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.

Reference to Existing Issue

Closes #169.

Description of Changes

  • Updated the real-time update logic to include address field updates via WebSocket.
  • Modified the listenForLocationUpdates function to handle and broadcast address changes.

Issues

  • Inheriting from ChannelLiveServerTestCase is causing test failures in Django 5.2, which is a new release, and a fix has yet to be released for it.

  • Adding ChannelLiveServerTestCase prevents running tests in parallel with the --parallel flag, as it relies on live WebSocket connections that cannot run concurrently.

Demo

real-time-address.webm

dee077 avatar Apr 27 '25 17:04 dee077