jaiarobotics / jaiabot

Jaiabot source code
Other
20 stars 2 forks source link

task/add-task-packets-to-rest-api-jaia-1517 #939

Closed edsanville closed 1 month ago

edsanville commented 1 month ago

Implemented the task packet API endpoint.

tsaubergine commented 1 month ago

I changed the base branch to https://github.com/jaiarobotics/jaiabot/tree/feature/app-for-api/jaia-1460 to simplify review of this PR (as it builds on that work).

edsanville commented 1 month ago

Please add filtering by jaia_request.target - Right now I always get all the task packets, whereas if I send, for example:

http://localhost:9092/jaia/v1/task_packets/b1?start_date=1722177986720894&end_date=1722350786720894

I just want task packets from bot 1

Whoops, yes this was an oversight.

tsaubergine commented 1 month ago

Merging into https://github.com/jaiarobotics/jaiabot/pull/925; final review can happen there.