Skip to content

Add webhook feature#272

Merged
DirtyRacer1337 merged 2 commits intoThePornDatabase:mainfrom
sigitosh:main
Mar 28, 2025
Merged

Add webhook feature#272
DirtyRacer1337 merged 2 commits intoThePornDatabase:mainfrom
sigitosh:main

Conversation

@sigitosh
Copy link
Copy Markdown
Contributor

Webhook Notification Feature

Adds webhook notifications when files are successfully renamed. External systems can now be notified via HTTP POST containing the renamed file path.

Key Features

  • Configurable webhook URL in namer.cfg
  • Simple enable/disable toggle
  • Error handling to prevent interruption of renaming workflow

Configuration

[webhook]
webhook_enabled = True
webhook_url = http://mediaserver/xyz

Includes full test coverage for all scenarios.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 28, 2025

Codecov Report

Attention: Patch coverage is 95.83333% with 3 lines in your changes missing coverage. Please review.

Project coverage is 85.10%. Comparing base (77994aa) to head (c78f5a4).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
test/namer_webhook_test.py 94.54% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #272      +/-   ##
==========================================
+ Coverage   84.91%   85.10%   +0.18%     
==========================================
  Files          40       41       +1     
  Lines        4647     4719      +72     
  Branches      522      525       +3     
==========================================
+ Hits         3946     4016      +70     
- Misses        421      422       +1     
- Partials      280      281       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@DirtyRacer1337 DirtyRacer1337 merged commit 8db2ada into ThePornDatabase:main Mar 28, 2025
14 checks passed
killershubham pushed a commit to killershubham/namer that referenced this pull request Dec 6, 2025
* add webhook feature

* refactor

---------

Co-authored-by: sigitosh <¨sigitosh@proton.me¨>
Co-authored-by: DirtyRacer <DirtyRacer@ya.ru>
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