If we get a 301/302 response, let's not do a full page load. Let's just reload the #page fragment.

marking as p5 - this doesn't sound like a blocker to me, let me know if that's incorrect

This affects breadcrumbs and causes extra load on mobile.

This is pretty important for us to have a slick, responsive app experience. It's also really hard. I'll keep poking at it.

What is left here? ETA? Importance?

The full implementation is left, essentially :) The difficulty is that we frequently redirect after a POST, and ajax has a very hard time (pretty much impossible) to follow a 30x response. Been chatting with cvan about possible solutions, but nothing that's made it to implementation yet.

Oh yeah I think the latest idea we were toying with was adding middleware that does something like if request.is_ajax() and request.method == 'POST' and response.status_code in (301, 302): return HttpResponse(response, json.dumps({'redirect': location}) and then have fragments.js handle that response and do a fragment refresh.

This seems like an important bug to fix. We need an owner for this.

Potch and I also discussed how also should work on an expiry for the header so we can do synchronous page refreshes if the header is out of date (after pushes for example).

(In reply to Chris Van Wiemeersch [:cvan] from comment #9) > Potch and I also discussed how also should work on an expiry for the header > so we can do synchronous page refreshes if the header is out of date (after > pushes for example). Cvan, if you can build out the middleware for this, I'd be happy to wire up fragments.js to handle it. What say ye?

(In reply to Potch [:potch] from comment #10) > (In reply to Chris Van Wiemeersch [:cvan] from comment #9) > > Potch and I also discussed how also should work on an expiry for the header > > so we can do synchronous page refreshes if the header is out of date (after > > pushes for example). > > Cvan, if you can build out the middleware for this, I'd be happy to wire up > fragments.js to handle it. What say ye? I'm totes down!

she's landed! https://github.com/mozilla/zamboni/commit/91d2d4e Thanks to cvan for the middleware!