Set cookie path to URL of homepage

[mgax]

Fixes #46

[tomusher]

Looks good to me. Are there any cases you can think of where the path of a root page is not /, or is this for future proofing?

[mgax]

Are there any cases you can think of where the path of a root page is not /, or is this for future proofing?

Future-proofing, in case somebody hosts a couple of Wagtail websites on the same domain. Though I'm wondering if the approach is right, because it introduces 2 database queries for each page load. Perhaps a better approach is to pass a site argument to the inclusion tag, and only if that's present, compute the homepage's URL, otherwise just use /?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (31909a4) 87.84% compared to head (d84e76f) 87.84%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   87.84%   87.84%           
=======================================
  Files          15       15           
  Lines         732      732           
  Branches      137      137           
=======================================
  Hits          643      643           
  Misses         54       54           
  Partials       35       35           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tomusher]

Great, that change makes sense to me!

[Stormheg]

Thank you! Just setting this to the root path sounds fine to me.