Skip to content
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

Fixed #13256 - Added option to switch to localStorage instead of cookies #13302

Merged
merged 2 commits into from
Jul 12, 2023

Conversation

snipe
Copy link
Owner

@snipe snipe commented Jul 12, 2023

This adds the ability to switch from cookieStorage to localStorage or sessionStorage in bootstrap table, which should be useful for folks with a LOT of custom fields hidden.

I set it to default to cookieStorage so for most people it won't break their existing remembered items.

@what-the-diff
Copy link

what-the-diff bot commented Jul 12, 2023

PR Summary

  • Addition to .env.example file
    A new environment variable BS_TABLE_STORAGE has been defined. It's set to cookieStorage, indicating the type of storage mechanism used, possibly for saving user preferences or sessions.

  • New configuration in config/session.php file
    A configuration option bs_table_storage was introduced. This corresponds to the mechanism for storing session information for users, as defined with the previously mentioned environment variable.

  • Updates to resources/views/partials/bootstrap-table.blade.php file
    The session configuration is now being used in the file responsible for rendering bootstrap tables on the front-end. The change suggests the session's storage method is now central to the table's functionality, probably affecting how user interactions with the table are stored and recalled.

@snipe snipe merged commit 232e84a into develop Jul 12, 2023
@snipe snipe deleted the features/env_setting_for_bs_tables branch July 12, 2023 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant