You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This ticket is from a discussion with @vinniefalco who was asking why the URL structures are different, why there's a query param in there instead of it being part of the URL. This is something that bugged me too when working on it, we can simplify the views and the shared code by unifying it to all being path based.
e.g.
/libraries/release/grid
/libraries/release/mini
/libraries/1.80.1/mini
/libraries/release/algorithm
The text was updated successfully, but these errors were encountered:
@daveoconnor the current cookie strategy include library_view and boost_version. This is working. It was after having sessions variables, and then removing session variables. which are not cdn-cachable. At least from the description of this issue 1360, the cookies can still be used.
Thanks @sdarwin ,yeah, that's all working fine, and I don't think it necessarily needs to change, it was just something that came up in the discussion at the time because having to account for the cookie (or session variable has the same issue) is a cause of some of the edge case urls Vinnie wasn't liking and should be noted. Shouldn't be an issue if we switch to the new urls above.
This ticket is from a discussion with @vinniefalco who was asking why the URL structures are different, why there's a query param in there instead of it being part of the URL. This is something that bugged me too when working on it, we can simplify the views and the shared code by unifying it to all being path based.
e.g.
/libraries/release/grid
/libraries/release/mini
/libraries/1.80.1/mini
/libraries/release/algorithm
The text was updated successfully, but these errors were encountered: