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
Release/scientific salamander #231
Merged
Merged
Conversation
This file contains 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
Also onChangeUseResult > onSubmitUseResult and onChangeStatus > submitStatus The reason is because the action is being referred to as submit everywhere else. Even the function that triggers the submit is called "submit". This was called onChange, because according to the internal useParcelState hook it is just a change, but the name should really reflect how its used
frameMetas job is to hold data that should only exist until the next change occurs (the current parcel frame) lastOriginId requires that same data lifespan lastOriginId was a concept that only react-dataparcels was interested in, and probably shouldnt have been put in plain dataparcels
Feature/on submit
replacing the inner parcel from props is intended to be a complete reset, so the buffer should also be emptied
when no rebasing and new data is received from props, buffer is wiped and buffers state is replaced with data from props when rebasing and new data is received from props, buffers state is replaced with data from props with buffer applied to it also fixed a wrong test that was relying on incorrect cachebehaviour in a change request
Refactor lastOriginId and add mergeMode
allanhortle
approved these changes
Jul 16, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Contains #229 #230
Addresses the first half of #123
dataparcels
dispatch()
when it should_lastOriginId
, replace with more genericframeMeta
react-dataparcels
useParcelForm onChange
is nowuseParcelForm onSubmit
useParcelForm onChangeUseResult
is nowuseParcelForm onSubmitUseResult
useParcelForm
's returnedonChangeStatus
is nowsubmitStatus
useParcelBuffer
(and thereforeuseParcelForm
) to rebase changes whenframeMeta.mergeMode === "rebase"