[Canvas] Fix by-reference embeddable migration error during workpad migration (#133911)

* Adds check in embeddable fn migration to migrate only by value embedables

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
This commit is contained in:
Catherine Liu 2022-06-09 16:33:11 -07:00 committed by GitHub
parent 1279650320
commit 346840b032
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -59,10 +59,12 @@ export function embeddableFunctionFactory({
const embeddableInput = decode(state.arguments.config[0] as string);
const embeddableType = state.arguments.type[0];
const migratedInput = migrateFn({ ...embeddableInput, type: embeddableType });
state.arguments.config[0] = encode(migratedInput);
state.arguments.type[0] = migratedInput.type as string;
if (embeddableInput.explicitInput.attributes || embeddableInput.explicitInput.savedVis) {
const migratedInput = migrateFn({ ...embeddableInput, type: embeddableType });
state.arguments.config[0] = encode(migratedInput);
state.arguments.type[0] = migratedInput.type as string;
}
return state;
};