Skip to content

Fix set_characteristic #140#143

Merged
ikalchev merged 1 commit intodevfrom
fix_set_chars
Sep 3, 2018
Merged

Fix set_characteristic #140#143
ikalchev merged 1 commit intodevfrom
fix_set_chars

Conversation

@ikalchev
Copy link
Copy Markdown
Owner

The accessory should respond either with 204 (NO_CONTENT) or
with 207 and json with a specific status code. This change
returns 204 for all characteristics.

The accessory should respond either with 204 (NO_CONTENT) or
with 207 and json with a specific status code. This change
returns 204 for all characteristics.
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #143 into dev will increase coverage by 0.07%.
The diff coverage is 5.55%.

@@            Coverage Diff             @@
##              dev     #143      +/-   ##
==========================================
+ Coverage   52.08%   52.16%   +0.07%     
==========================================
  Files          15       15              
  Lines        1342     1342              
  Branches      138      137       -1     
==========================================
+ Hits          699      700       +1     
+ Misses        628      627       -1     
  Partials       15       15
Impacted Files Coverage Δ
pyhap/accessory_driver.py 58.49% <0%> (+1.13%) ⬆️
pyhap/hap_server.py 26.02% <7.69%> (-0.07%) ⬇️

@ikalchev ikalchev merged commit 6272c7b into dev Sep 3, 2018
@ikalchev ikalchev deleted the fix_set_chars branch September 3, 2018 18:09
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