Skip to content

Conversation

@palazzem
Copy link
Owner

Related Issues

Proposed Changes:

This change (re)introduces the web page login when e-Connect alarm system is used. This is required as it seems the public API doesn't register the client to get updates from the long-polling API.

We introduce also tests/scripts/debug_polling.py debug script to verify if long-polling is working or not in a real world scenario.

Testing:

Use tests/scripts/debug_polling.py to see if long polling works in your scenario.

Extra Notes (optional):

This is a temporary change as it looks like Elmo is making updates to the Cloud API. As soon as a new public and documented API is out, we'll probably drop this approach in favor of something more reliable.

Checklist

  • Related issues and proposed changes are filled
  • Tests are defining the correct and expected behavior
  • Code is well-documented via docstrings

@palazzem palazzem added this to the 0.12.2 milestone Apr 13, 2025
@palazzem palazzem self-assigned this Apr 13, 2025
@palazzem palazzem moved this to Waiting for Feedback in e-Connect Roadmap Apr 13, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 14430103988

Details

  • 21 of 21 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 14429372916: 0.0%
Covered Lines: 417
Relevant Lines: 417

💛 - Coveralls

@palazzem palazzem merged commit ebc6d33 into main Apr 13, 2025
4 checks passed
@palazzem palazzem deleted the palazzem/fix-158-2 branch April 13, 2025 13:54
@github-project-automation github-project-automation bot moved this from Waiting for Feedback to Done in e-Connect Roadmap Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

Polling is not triggered even when there are changes

3 participants