-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Table Column Cookies are set to expire on browser close #5014
Comments
The expiration date is set to 2 years |
Yes, that seems to be the expected behavior. However as you can see in my screenshot, the cookie that is set does not expire in two years. Rather, it expires once the browser session has ended. |
In my brief testing, it appears that the cookie is only set if you make a change to the columns. For example, if you go to https://demo.snipeitapp.com/hardware and add/remove any of the options, the "hardware.index.bs.table.columns" cookie is then set. If you go to /users, the "users.index.bs.table.columns" cookie is set. It appears to set one for every table that is modified, all of which expire once the browser closes. |
Yeah, I was testing locally by turning columns off and on. I did see this issue in the bootstrap tables library, but I haven't seen others complaining about this on their repo. |
Okay, I think I have this working locally. Still testing. Will check back. |
No, as I said, I almost have a fix for it. Was a bug in the JS library, I think |
Expected Behavior (or desired behavior if a feature request)
When selecting/deselecting column preferences, the options set are stored in a cookie. When the browser closes, Snipe reads the cookie and restores column preferences.
Actual Behavior
The cookie is set during the current browser session, but is set to expire once that browsing session ends. Upon closing and reopening any browser, the column preferences are reverted back to defaults.
Please confirm you have done the following before posting your bug report:
Provide answers to these questions:
storage/logs
and your webserver's logs.Please do not post an issue without answering the related questions above. If you have opened a different issue and already answered these questions, answer them again, once for every ticket. It will be next to impossible for us to help you.
https://snipe-it.readme.io/docs/getting-help
Screenshot of cookie in Google Chrome (Note expiry date at the bottom)
The text was updated successfully, but these errors were encountered: