From fb9afd32db582cda6cebfbb84c93c0aa2b4bbb03 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Wed, 17 Aug 2016 10:02:50 +0100 Subject: [PATCH 1/1] Make banner cookies expire after one year This avoids having lots of cookies stick around forever and we shouldn't ever have a banner up for that long. --- app/assets/javascripts/index.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/assets/javascripts/index.js b/app/assets/javascripts/index.js index 9ebee2dd2..782b9363c 100644 --- a/app/assets/javascripts/index.js +++ b/app/assets/javascripts/index.js @@ -174,12 +174,15 @@ $(document).ready(function () { $.cookie('_osm_welcome', 'hide', { expires: expiry }); }); + var bannerExpiry = new Date(); + expiry.setYear(expiry.getFullYear() + 1); + $('#banner .close-wrap').on('click', function(e) { var cookieId = e.target.id; $('#banner').hide(); e.preventDefault(); if (cookieId) { - $.cookie(cookieId, 'hide', { expires: expiry }); + $.cookie(cookieId, 'hide', { expires: bannerExpiry }); } }); -- 2.43.2