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
{{ message }}
This repository has been archived by the owner on Nov 3, 2020. It is now read-only.
Currently when you use the Cookie module and try to set a cookie with days parameter it must be an Integer, not a decimal. If it is a decimal it fails silently and defaults to session length.
To Reproduce
Steps to reproduce the behavior:
Require the Cookie module
Call the cookie set function with some dummy data and a decimal days parameter: Cookie.set('test', 'test', { days: 10.5 });
Check that the cookie length time defaults to session time on your browser cookie panel.
Expected behavior
Alert the user that decimals can't be used or round the number up or down instead of defaulting to session length
Describe the bug
Currently when you use the Cookie module and try to set a cookie with days parameter it must be an Integer, not a decimal. If it is a decimal it fails silently and defaults to session length.
To Reproduce
Steps to reproduce the behavior:
set
function with some dummy data and a decimal days parameter:Cookie.set('test', 'test', { days: 10.5 });
Expected behavior
Alert the user that decimals can't be used or round the number up or down instead of defaulting to session length
Additional context
Related issues:
The text was updated successfully, but these errors were encountered: