Skip to content
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

Improve lazy loading of Monaco editor and fix webworker warning #220

Merged
merged 1 commit into from Mar 20, 2024

Conversation

martin-fleck-at
Copy link
Contributor

  • Load web worker through blob URL
  • Avoid scripts from being loaded automatically if they are dynamic

Requires axonivy/process-editor-client#515

Copy link
Member

@ivy-lli ivy-lli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. @ivy-lmu ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool thanks!

@ivy-lli
Copy link
Member

ivy-lli commented Mar 20, 2024

@martin-fleck-at new version of the process editor packages should be in the npm repo

@martin-fleck-at
Copy link
Contributor Author

@ivy-lli Thank you, I'll update this PR very soon!

Copy link
Member

@ivy-lmu ivy-lmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thanks

@ivy-lli
Copy link
Member

ivy-lli commented Mar 20, 2024

@martin-fleck-at Did you update this PR? Because it seems to work now :) Can you maybe update the @axonivy dependencies in the package.json/.lock?

@ivy-lmu
Copy link
Member

ivy-lmu commented Mar 20, 2024

@ivy-lli I have triggered the jobs

- Load web worker through blob URL
- Avoid scripts from being loaded automatically if they are dynamic

Co-authored-by: Olaf Lessenich <olessenich@eclipsesource.com>
@martin-fleck-at
Copy link
Contributor Author

@ivy-lli @ivy-lmu Unfortunately, my internet was out for some time. I pushed the update now, sorry for the delay!

@martin-fleck-at
Copy link
Contributor Author

I only updated the yarn.lock, the rest is still the same.

@ivy-lli
Copy link
Member

ivy-lli commented Mar 20, 2024

The PR looks good and the builds are happy. I will merge :)

@ivy-lli ivy-lli merged commit 5bfb405 into axonivy:master Mar 20, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants