search

senseobservationsystems / commonsense-python-lib

Library for using CommonSense API in Python applications
Apache License 2.0
4 stars 1 forks source link

DELETE call failed #23

Closed yulrizka closed 10 years ago

yulrizka commented 10 years ago

When I'm testing updating plan on TBS (deleting & inserting new data point), I found out the the commonsense return 404 when deleting the sensor data by id

Recent commit might change something that breaks the functionality to send DELETE on certain API call.

I also notice that not every test pass when i run SenseAPITest.py

example:

this is deleting sensor data by id on master d97e10a924f details removed

yulrizka commented 10 years ago

/cc @Tubyhes @tedschmidt

tedschmidt commented 10 years ago

The call is wrong, sensor instead of sensors: http://api.sense-os.nl/sensor/405333/data/522dbd06b4b735be05002431.json should be: http://api.sense-os.nl/sensors/405333/data/522dbd06b4b735be05002431.json /cc @yulrizka

yulrizka commented 10 years ago

Oh my.. why didn't i saw that.. nice catch @tedschmidt , woring on patch right now