search

jonathan-s / django-sockpuppet

Build reactive applications with the django tooling you already know and love.
https://github.com/jonathan-s/django-sockpuppet
MIT License
450 stars 22 forks source link

Html should take the innerhtml to be correct #20

Closed jonathan-s closed 4 years ago

jonathan-s commented 4 years ago

When making a partial dom update it's more important that it's using innerhtml. As per this answer using decode_contents is the correct approach for beautifulsoup.

codecov-commenter commented 4 years ago

Codecov Report

Merging #20 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master     #20   +/-   ##
======================================
  Coverage    3.05%   3.05%           
======================================
  Files          12      12           
  Lines         262     262           
  Branches       27      27           
======================================
  Hits            8       8           
  Misses        254     254
Impacted Files Coverage Δ
sockpuppet/consumer.py 0.00% <ø> (ø)

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 d2623f4...3c7c721. Read the comment docs.