magento / magento2-page-builder

Magento2 PageBuilder
Other
80 stars 62 forks source link

Remove JavaScript Source Maps #798

Open Scarbous opened 2 years ago

Scarbous commented 2 years ago

Please remove //# sourceMappingURL= form generated JS Files or add the map files.

Because it leads to the following behavior:

When I use my DevTools tons of errors like: DevTools failed to load source map: Could not load content for ....js.map: HTTP error: status code 404, net::ERR_HTTP_RESPONSE_CODE_FAILURE

This leads to the following error:

actions.js:105 Uncaught TypeError: Cannot read properties of undefined (reading 'map')
    at UiClass.updateActions (actions.js:105)
    at setNested (objects.js:44)
    at Object.nested (objects.js:118)
    at UiClass.set (element.js:305)
    at updateValue (links.js:80)
    at events.js:86
    at Array.forEach (<anonymous>)
    at trigger (events.js:83)
    at UiClass.trigger (events.js:161)
    at UiClass.<anonymous> (element.js:424)

I use Chrome, when I disable Enable JavaScript source maps in the DevTools Settings the error is gone.

Preconditions (*)

Steps to reproduce (*)

Expected result (*)

Edit Page and save changes without getting logged out.

Actual result (*)

When I access the CMS-Page Edit site with open Chrome-DevTools I get logged out, when I edit the page content with PageBuilder.

m2-assistant[bot] commented 2 years ago

Hi @Scarbous. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


setrada commented 2 years ago

Having nearly the same result on M.2.4.4 with PageBuilder 2.2.2. Lot of sourcemap files are not generated at the adminhtml.

kamephis commented 1 year ago

Same here, M 2.4.4 / 2.2.2