Merge pull request #355 from dabapps/fix-cookie-path

Specify cookie for entire subdomain rather than specific subpath
dabapps · Apr 23, 2021 · 9ca6a0d · 9ca6a0d
2 parents 03e2ad7 + 38b0300
commit 9ca6a0d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@dabapps/roe",
-  "version": "0.13.1",
+  "version": "0.13.2",
   "description": "A collection of React components, styles, mixins, and atomic CSS classes to aid with the development of web applications.",
   "main": "dist/js/index.js",
   "types": "dist/js/index.d.ts",

diff --git a/src/ts/components/banners/cookie-banner.tsx b/src/ts/components/banners/cookie-banner.tsx
@@ -46,6 +46,7 @@ const CookieBanner = (props: CookieBannerProps) => {
   const setCookie = () => {
     document.cookie = cookie.serialize('cookies-accepted', 'true', {
       maxAge: props.maxAge || A_YEAR_IN_SECONDS,
+      path: '/',
     });
     setDismissed(true);
   };