9001 / 9001/copyparty

feature request: hook effects in xiu hooks

Đang mở
#1,571 0 bình luận 1 reaction 1 người được giao Được @9001 nhận Xem trên GitHub
enhancement
Ngôn ngữ chính
Python
Star
46.7k
Fork
1.9k
Merge trung bình
9 ngày 2 giờ
Pull request đã merge (30 ngày)
4

Mô tả

**is your feature request related to a problem? Please describe.**
I'm making a hook that [reencodes FLAC to Opus on upload](https://github.com/minipps/copyparty-hooks/blob/main/lossless_to_opus.py). For this I chose xiu since I want to do this after full file uploads and I want to be able to delete the old FLAC and re-index the new OGG after upload, however, it seems like xiu uses `runihook` instead of `runhook` which doesn't support hook effects

**Describe the idea / solution you'd like**
Add [hook effects](https://github.com/9001/copyparty/blob/fface524c3463918677bd01a6aff7d09dbbbc29d/docs/devnotes.md#hook-effects) to xiu too :)

**Describe any alternatives you've considered**
for now I'm just using a lower scan period for indexing and os.unlink() to delete the file manually from inside my hook, although this is obviously way worse.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Xem module xiu để hiểu hiện tại nó sử dụng runihook như thế nào. So sánh với implementation runhook thông thường có hỗ trợ hook effects. Liên kết devnotes.md giải thích về hook effects. Thay đổi này có thể sẽ liên quan đến việc sửa xiu để gọi runhook với hỗ trợ effects, sau đó kiểm thử bằng một hook mẫu sử dụng các effects như xóa tệp hoặc lập chỉ mục lại.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Lĩnh vực
backend, cli
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Đặc tả rõ ràng
Mức phù hợp với người mới
65/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.