- Aug 20, 2015
-
-
Martin Burchell authored
-
Martin Burchell authored
-
Martin Burchell authored
-
- Aug 17, 2015
-
-
Mark Skipper authored
We got bad performance degrading as the number of terms increased. Enabling django_debug_toolbar it was easy to see a bunch of queries getting terms related to items duplicated as many times as items in the system. Making the api view for listing items prefetch related terms (and their taxonomies) removes these O(n) queries
-
Alice Heaton authored
-
Martin Burchell authored
-
Martin Burchell authored
The next hidden parameter in the form, which determines the redirect following an update is set to the 'next' query parameter in the form. If the query parameter does not exist, the HTTP_REFERER is used. If neither exist, the user gets redirected to the dashboard.
-
Alice Heaton authored
-
Alice Heaton authored
Add the add item functionality. Also enforce a default value for item-type on forms. While the API allows for items without an item-type, the form's handling of it was inconsistant. Until we have a use case for items without an item type, a default is enforced.
-
- Aug 14, 2015
-
-
Martin Burchell authored
-
Alice Heaton authored
-
Martin Burchell authored
-
Alice Heaton authored
-
Alice Heaton authored
-
Martin Burchell authored
-
Martin Burchell authored
-
Martin Burchell authored
-
Alice Heaton authored
-
Martin Burchell authored
-
Alice Heaton authored
-
Martin Burchell authored
-
Alice Heaton authored
Make the update button first in the code so that html5 browsers consider it the default button when pressing enter.
-
Martin Burchell authored
-
Alice Heaton authored
-
Martin Burchell authored
This is a temporary fix - we should fix this properly by getting the serializer to format datetimes for us.
-
Alice Heaton authored
-
- Aug 13, 2015
-
-
Alice Heaton authored
-
Alice Heaton authored
-
Martin Burchell authored
-
Martin Burchell authored
-
Mark Skipper authored
Disabling the API urls is a quick fix security measure. Obviously we'd want the api urls to exist -- with suitable authentication in a real system. And it sucks to loose the ability to use the web interface to the API that DRF provides during testing, so here I'm disabling only if DEBUG=False
-
Mark Skipper authored
The API urls don't need to appear in the system urls. They code that uses the api, including the transport layer, can function without it. This commit removes all the calculation of correct urls using reverse (which depends on the ruls being included in the systme urls) and replaces with just spaces -- which seems to work just fine.
-
Alice Heaton authored
-
Martin Burchell authored
Need to figure out how to get the read-only properties serialized.
-
Mark Skipper authored
Responding to feedback from code review, change the item fixture to use the create method from transport.items instead of FactoryBoy
-
Alice Heaton authored
-
Mark Skipper authored
If you call get with an invalid item id, you should get a TransportException that communicates the nature of the failure Added test and updated body of get to raisae exception on unknown item
-
Martin Burchell authored
-
Mark Skipper authored
-
Mark Skipper authored
Added a simple test for the happy day scenario of getting an item from /api/items/nn/ and made the test pass by calling the API
-