search

wagtail / wagtail-live

High speed publishing from messaging apps to a Wagtail live blog. A GSoC 2021 project.
https://wagtail.github.io/wagtail-live/
BSD 3-Clause "New" or "Revised" License
37 stars 6 forks source link

Gracefully handle error when deleting a missing message #142

Closed Tijani-Dia closed 2 years ago

Tijani-Dia commented 2 years ago

This PR aims to fix issue #141.

When a KeyError is raised when deleting a missing message, warn the user and return a 200 response.

codecov-commenter commented 2 years ago

Codecov Report

Merging #142 (6cee4ab) into main (dd769be) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #142   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           40        40           
  Lines         1091      1094    +3     
  Branches       103       149   +46     
=========================================
+ Hits          1091      1094    +3
Impacted Files Coverage Δ
src/wagtail_live/receivers/base.py 100.00% <100.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 dd769be...6cee4ab. Read the comment docs.