search

wlanslovenija / PiplMesh

A social info portal for wireless networks.
http://dev.wlan-si.net/wiki/PiplMesh
Other
40 stars 19 forks source link

Issue #301 - Support marking notifications as read #309

Closed zupan closed 11 years ago

zupan commented 11 years ago

Notifications can be marked as read. #301

zupan commented 11 years ago

There is an error in Tastypie when using PATCH on resource where a related field has full=True parameter. This is known issue and it was fixed here.

zupan commented 11 years ago

This solution seems to be working.

mitar commented 11 years ago

Great! I pushed change to my Tastypie fork. It is now in requirements.txt. Thanks for finding this out.

zupan commented 11 years ago

Problem with related fields is still there. Hope to find solution in tests.

mitar commented 11 years ago

Have you found error through tests?

mitar commented 11 years ago

So the test you made is failing?

mitar commented 11 years ago

All tests pass for me? (8 tests in 97.395s) You have more tests? Something fails?

zupan commented 11 years ago

Tests are finally breaking at the same error that causing problem in application when trying to mark notification as read.

mitar commented 11 years ago

Tests are finally breaking at the same error that causing problem in application when trying to mark notification as read.

Fixed. Please update from main repository.

zupan commented 11 years ago

Everything is working and up to date.

mitar commented 11 years ago

I merged it.