search

mdn / kuma

The project that powers MDN.
https://developer.mozilla.org
Mozilla Public License 2.0
1.93k stars 679 forks source link

feat(notifications): process Content Updates via /update/ route #8070

Closed caugner closed 2 years ago

caugner commented 2 years ago

Part of https://github.com/mdn/yari-private/issues/981.

Before

  1. /update/ created only BCD Update Notifications.
  2. /create/pr/ created a single Content Update Notification independently.

After

  1. /update/ creates both BCD and Content Update Notifications.
  2. /create/pr/ creates a single Content Update Notification by reusing the same logic.
codecov-commenter commented 2 years ago

Codecov Report

Merging #8070 (bfe08a6) into main (57285dc) will increase coverage by 0.47%. The diff coverage is 78.33%.

@@            Coverage Diff             @@
##             main    #8070      +/-   ##
==========================================
+ Coverage   79.87%   80.35%   +0.47%     
==========================================
  Files          87       88       +1     
  Lines        3269     3303      +34     
  Branches      387      388       +1     
==========================================
+ Hits         2611     2654      +43     
+ Misses        598      583      -15     
- Partials       60       66       +6
Impacted Files Coverage Δ
kuma/notifications/utils.py 38.46% <60.71%> (+20.09%) :arrow_up:
kuma/api/v1/plus/notifications.py 57.48% <80.00%> (+2.50%) :arrow_up:
kuma/api/tests/test_admin.py 100.00% <100.00%> (ø)
kuma/api/v1/auth.py 60.00% <0.00%> (+2.22%) :arrow_up:

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 57285dc...bfe08a6. Read the comment docs.