Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add detection of new devices #524

Closed
wants to merge 14 commits into from
Closed

Add detection of new devices #524

wants to merge 14 commits into from

Conversation

bouwew
Copy link
Contributor

@bouwew bouwew commented Feb 29, 2024

No description provided.

@bouwew bouwew requested a review from CoMPaTech February 29, 2024 07:03
Copy link

codecov bot commented Feb 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (9ac599a) to head (2289b96).

❗ Current head 2289b96 differs from pull request most recent head 78b306f. Consider uploading reports for the commit 78b306f to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #524   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines         3149      3175   +26     
=========================================
+ Hits          3149      3175   +26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bouwew bouwew force-pushed the config-change branch 2 times, most recently from 502b163 to db4944e Compare March 18, 2024 07:17
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@bouwew bouwew force-pushed the config-change branch 2 times, most recently from b046bf7 to 8fc749e Compare April 19, 2024 14:40
And add to output
@bouwew bouwew force-pushed the config-change branch 2 times, most recently from e405743 to 901adee Compare April 26, 2024 17:31
Copy link
Member

@CoMPaTech CoMPaTech left a comment

Choose a reason for hiding this comment

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

👍

Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@bouwew
Copy link
Contributor Author

bouwew commented May 10, 2024

Closed, solved in pw-beta/Core Plugwise.

@bouwew bouwew closed this May 10, 2024
@bouwew bouwew deleted the config-change branch May 10, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants