-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
[DTS] Only delete types of data that are being transferred #17730
Conversation
…cts-also-on-restore
packages/core/data-transfer/src/strapi/providers/local-destination/index.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, tested it as well
packages/core/data-transfer/src/strapi/providers/local-destination/index.ts
Outdated
Show resolved
Hide resolved
packages/core/data-transfer/src/strapi/providers/local-destination/index.ts
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What does it do?
Update the import and transfer commands to generate the correct restore object
TODO:
refactor restore logic so it's modular and independent from local-sourceI'm skipping this one because it would be a big structural change that might break things*Why is it needed?
The initial functionality of the restore strategy causing a global reset of Strapi data caused too many users to be confused.
The updated functionality of only touching data included in a transfer matches user expectations to not touch files that are excluded in a transfer or import.
How to test it?
Using exclude or only with transfer or import should now only delete the files that are included in the process.
Related issue(s)/PR(s)
Closes #17572
Completes DX-862, DX-863, DX-864