Skip to content

Add Nightlight mode, extract deprecated nightlight fade from mode#315

Merged
frenck merged 1 commit intomainfrom
frenck-2021-1735
Jun 12, 2021
Merged

Add Nightlight mode, extract deprecated nightlight fade from mode#315
frenck merged 1 commit intomainfrom
frenck-2021-1735

Conversation

@frenck
Copy link
Copy Markdown
Owner

@frenck frenck commented Jun 12, 2021

Proposed Changes

The Nightlight fade property has been deprecated and will be removed from WLED 0.13.
Additionally, Nightlight mode was introduced replacing fade.

This PR implements the new mode property and re-implements the fade property by getting it from mode in case it isn't present in the API (and vice versa for backward compatibility).

@frenck frenck added the new-feature New features or options. label Jun 12, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 12, 2021

Codecov Report

Merging #315 (8d9296a) into main (a15fb0d) will increase coverage by 0.25%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #315      +/-   ##
==========================================
+ Coverage   67.03%   67.28%   +0.25%     
==========================================
  Files           4        4              
  Lines         364      376      +12     
  Branches       70       72       +2     
==========================================
+ Hits          244      253       +9     
- Misses        109      110       +1     
- Partials       11       13       +2     
Impacted Files Coverage Δ
src/wled/models.py 80.11% <75.00%> (-0.37%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a15fb0d...8d9296a. Read the comment docs.

@frenck frenck merged commit b0f319d into main Jun 12, 2021
@frenck frenck deleted the frenck-2021-1735 branch June 12, 2021 15:42
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

new-feature New features or options.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant