Fix orders not showing up in admin. (#153)

This change anchors the longclaworders API URLs with a '^' to ensure they can't
confusingly match 'ANYTHING/api/order/...'. Also forces the order loading
JavaScript to use an absolute URL.

I'm not sure that this is the best architectural solution, but it resolves the
issues I'm having at least.

Fixes #131.
pull/155/head
BenSturmfels 2018-03-24 05:56:57 +11:00 zatwierdzone przez James Ramm
rodzic e0078aa6d7
commit f498ac084c
2 zmienionych plików z 5 dodań i 4 usunięć

Wyświetl plik

@ -7,7 +7,7 @@
<p class="back"><a href="{{ view.index_url }}" class="icon icon-arrow-left">{% blocktrans with view.verbose_name as model_name %}Back to {{ model_name }} list{% endblocktrans %}</a></p>
<div id="order-app" data-order-id="{{ order_id }}" data-url-prefix="{{ api_url_prefix }}"></div>
<div id="order-app" data-order-id="{{ order_id }}" data-url-prefix="/{{ api_url_prefix }}"></div>
</div>
<script type="text/javascript" src="{% static 'longclawcore/js/vendors.bundle.js' %}"></script>
<script type="text/javascript" src="{% static 'longclawcore/js/orders.bundle.js' %}"></script>

Wyświetl plik

@ -15,21 +15,22 @@ refund_order = api.OrderViewSet.as_view({
'post': 'refund_order'
})
PREFIX = r'^{}order/'.format(API_URL_PREFIX)
urlpatterns = [
url(
API_URL_PREFIX + r'order/(?P<pk>[0-9]+)/$',
PREFIX + r'(?P<pk>[0-9]+)/$',
orders,
name='longclaw_orders'
),
url(
API_URL_PREFIX + r'order/(?P<pk>[0-9]+)/fulfill/$',
PREFIX + r'(?P<pk>[0-9]+)/fulfill/$',
fulfill_order,
name='longclaw_fulfill_order'
),
url(
API_URL_PREFIX + r'order/(?P<pk>[0-9]+)/refund/$',
PREFIX + r'(?P<pk>[0-9]+)/refund/$',
refund_order,
name='longclaw_refund_order'
)