ActivityWatch / ActivityWatch/aw-server-rust

DELETE endpoint to delete event(s)

未關閉
#60 7 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
enhancement
主要語言
Rust
星號
315
分支
96
平均合併
2 天 21 小時
30 天內合併 PR
14

描述

My suggested API:
- Remove `DELETE /events/` endpoint
- Add `POST /events/delete/by_ids`
- Add `GET /events/delete/by_period?start=X&end=X`

Reasons why the old delete API is bad and should be replaced completely:
- The new "by_ids" endpoint makes it possible to remove multiple specific events in one go while the old API could only delete one event at a time.
- The old DELETE endpoint is also broken because both `/events/` and `/events/count` are in the same namespace

Things which need to be considered:
- Vacuuming the database

Resources:
- Discussion with @leoschwarz in aw-server-python PR https://github.com/ActivityWatch/aw-server/pull/57

貢獻指南

這個儲存庫沒有索引到貢獻指南

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。