-
Notifications
You must be signed in to change notification settings - Fork 2
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
[Feature] Remove multiple frames #392
Conversation
Coverage ReportCoverage report can be checked at https://cdnepgrafxstudiodev.azureedge.net/sdk/coverage/392/coverage.html Use PR sdk packageTarball can be downloaded from https://cdnepgrafxstudiodev.azureedge.net/sdk/dev-packages/392/studio-sdk.tgz To use in local project, change package.json to use local tarball "dependencies": {
"@chili-publish/studio-sdk": "https://cdnepgrafxstudiodev.azureedge.net/sdk/dev-packages/392/studio-sdk.tgz"
} |
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.
Shouldn't we now redirect remove
to the new removeFrames
and completely remove the exposed removeFrame
on engine-side?
agree, it could be an option. not sure what others think? |
Agree 👍 |
3b4c9cc
…-remove' into feature/EDT-1187-multiple-frames-remove
This PR
PR Guidelines
Related tickets