-
Notifications
You must be signed in to change notification settings - Fork 430
feat(store)!: fully rework and add auto save #1550
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
Merged
Merged
Changes from 20 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
94ab85d
Add auto save to store plugin
Legend-Master 389e811
Merge branch 'v2' into store-auto-save
Legend-Master bb1db1c
Put jsdoc at constructor instead of class level
Legend-Master 7ffb3d5
Clippy
Legend-Master 2f91667
Use enum instead of bool
Legend-Master 4012005
Some(AutoSaveMessage::Cancel) | None
Legend-Master fbfcba8
from_millis
Legend-Master 571317a
u64
Legend-Master e12d3fc
Add change file
Legend-Master 7ce9e60
Rename to emit_on_change
Legend-Master b85c863
should use Duration in `with_store`
Legend-Master 5d35b1f
Add breaking change notice to change file
Legend-Master 2ee919b
Emit change event for inserts by reset
Legend-Master d18370d
Update readme example
Legend-Master 9336e7f
Update example
Legend-Master 35435ec
Remove extra line
Legend-Master 3d84eab
Make description clear it only works with managed
Legend-Master 23c9c21
Fix links in docstring
Legend-Master 22f11b9
Fix doc string closing
Legend-Master 346ea33
Merge remote-tracking branch 'upstream/v2' into store-auto-save
Legend-Master 738e0a8
Merge branch 'v2' into store-auto-save
Legend-Master f398859
get_mut
Legend-Master c2b0bd8
Proof of concept
Legend-Master 195384f
fmt
Legend-Master 5f808fd
Load store on create
Legend-Master 436c762
Merge branch 'v2' into store-auto-save
Legend-Master bdc80d5
cargo fmt
Legend-Master 5cadedf
Merge branch 'store-auto-save' of https://github.com/Legend-Master/pl…
Legend-Master 6287665
Merge branch 'v2' into store-auto-save
Legend-Master 7f99e47
Merge branch 'v2' into store-auto-save
Legend-Master 75e9f0c
Merge branch 'v2' into store-auto-save
Legend-Master 34dca14
Merge branch 'v2' into store-auto-save
Legend-Master ea351fb
Merge branch 'store-auto-save' of https://github.com/Legend-Master/pl…
Legend-Master 7d6dbc3
Fix merge conflits
Legend-Master 2ed4651
Merge branch 'v2' into store-auto-save
Legend-Master 7f43d5e
Merge branch 'v2' into store-auto-save
tweidinger ed0619a
Merge branch 'v2' into store-auto-save
Legend-Master 5a7d5aa
Merge branch 'store-auto-save' of https://github.com/Legend-Master/pl…
Legend-Master 9ee0b7b
Format
Legend-Master 456a0f5
Merge branch 'v2' into store-auto-save
Legend-Master 77bc7f0
Merge branch 'v2' into store-auto-save
Legend-Master d1ab707
Merge remote-tracking branch 'origin/v2' into store-auto-save
lucasfernog 578487a
small cleanup
lucasfernog 84351a3
update docs, use `impl Into<JsonValue>`
lucasfernog 5aaa591
fix doctests, further simplification of api
lucasfernog d8ae8b2
add store options
lucasfernog File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
--- | ||
"store": patch | ||
--- | ||
|
||
Add a setting `auto_save` to enable a store to debounce save on modification (on calls like set, clear, delete, reset) | ||
|
||
**Breaking change**: `with_store` now takes one more parameter `auto_save: Option<Duration>` | ||
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.