New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Please disable cookies! #2169

Open

centurianii opened this issue Jun 20, 2018 · 1 comment

centurianii commented Jun 20, 2018 •

edited

Loading

Your cookies (in plural) messes with my authentication cookies and it's a bad practice to achieve memory!

Have you noticed that after a while cookies are piled up?

Which lines should I go and delete?

Member

hikalkan commented Jun 22, 2018

You can override _setCookie & _getCookie functions to change cookie storage to local storage.
See https://github.com/hikalkan/jtable/blob/master/lib/jquery.jtable.js#L1246

Extending is easy. See an extension to understand how it works:

https://github.com/hikalkan/jtable/blob/master/lib/extensions/jquery.jtable.record-actions.js#L13

falwickster mentioned this issue

Extension method for cookies - replace cookies with local storage #2175

Merged

# for free to join this conversation on GitHub. Already have an account? # to comment