ActivityWatch / ActivityWatch/aw-server-rust

DELETE endpoint to delete event(s)

Aberta
#60 7 comentários 0 reações 0 responsáveis Ver no GitHub
enhancement
Linguagem predominante
Rust
Estrelas
315
Forks
96
Merge médio
2d 21h
PRs com merge (30d)
14

Descrição

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

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.