Make banner cookies expire after one year
authorTom Hughes <tom@compton.nu>
Wed, 17 Aug 2016 09:02:50 +0000 (10:02 +0100)
committerTom Hughes <tom@compton.nu>
Wed, 17 Aug 2016 09:02:50 +0000 (10:02 +0100)
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

index 9ebee2d..782b936 100644 (file)
@@ -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 });
     }
   });