Closed iruzevic closed 4 months ago
I wanted to test the Countries.php
part (there are several foreach statements that I think could be simplified a bit), but I got errors when I tried bundling the plugin.
ERROR in ./node_modules/@eightshift/frontend-libs/scripts/editor/css-variables.js
Module build failed (from ./node_modules/swc-loader/src/index.js):
Error: × Expression expected
╭─[/Users/dzoljom/Sites/infobip/wp-content/plugins/eightshift-forms/node_modules/@eightshift/frontend-libs/scripts/editor/css-variables.js:447:1]
444 │ }
445 │
446 │ // Output the style for CSS variables.
447 │ return <style dangerouslySetInnerHTML={{__html: `${output} ${finalManualOutput}`}} />;
· ─
448 │ };
449 │
450 │ /**
╰────
× Expected ';', got 'dangerouslySetInnerHTML'
╭─[/Users/dzoljom/Sites/infobip/wp-content/plugins/eightshift-forms/node_modules/@eightshift/frontend-libs/scripts/editor/css-variables.js:447:1]
444 │ }
445 │
446 │ // Output the style for CSS variables.
447 │ return <style dangerouslySetInnerHTML={{__html: `${output} ${finalManualOutput}`}} />;
· ───────────────────────
448 │ };
449 │
450 │ /**
╰────
Caused by:
Syntax Error
@ ./node_modules/@eightshift/frontend-libs/scripts/editor/index.js 6:0-100 6:0-100 6:0-100 6:0-100 6:0-100
@ ./src/Blocks/assets/scripts/blocks-editor.js 13:0-112 16:0-14 18:0-24 20:23-33
@ ./src/Blocks/assets/application-blocks-editor.js 11:0-33
ERROR in ./node_modules/@eightshift/frontend-libs/scripts/editor/editor.js
Module build failed (from ./node_modules/swc-loader/src/index.js):
Error: × Expression expected
╭─[/Users/dzoljom/Sites/infobip/wp-content/plugins/eightshift-forms/node_modules/@eightshift/frontend-libs/scripts/editor/editor.js:101:1]
98 │ } = props;
99 │
100 │ return (
101 │ <>
· ─
102 │ {OptionsComponent &&
103 │ <InspectorControls>
104 │ <OptionsComponent {...props} />
╰────
× Expression expected
...
There were a lot of these errors, and I couldn't bundle the plugin :/
Tnx @dingo-d for the suggestion. I have fixed the build problem. We still have problem with the eslint but this will be fixed soon. I have merged your PR and tested it. Looks ok
Changed
Removed
Added