Skip to content

Fix es6 commonjs mix#384

Merged
nkavtur merged 2 commits intomasterfrom
fix-es6-commonjs-mix
Feb 4, 2026
Merged

Fix es6 commonjs mix#384
nkavtur merged 2 commits intomasterfrom
fix-es6-commonjs-mix

Conversation

@nkavtur
Copy link
Contributor

@nkavtur nkavtur commented Feb 3, 2026

✏️ Changes

Minor fix for mixed usage of es6/commonjs modules.

📷 Screenshots

If there were visual changes to the application with this change, please include before and after screenshots here. If it has animation, please use screen capture software like to make a gif.

🔗 References

Include at at least one link to an explanation + requirements for this change, and more if at all possible. Typically this is a Jira/GitHub Issue, but could also be links to Zendesk tickets, RFCs, rollout plan or Slack conversations (for Slack conversations, make sure you provide a summary of the conversation under “Changes”).

🎯 Testing

✅ This change has been tested in a Webtask

✅ This change has unit test coverage

✅ This change has integration test coverage

✅ This change has been tested for performance

🚀 Deployment

✅ This can be deployed any time

🎡 Rollout

Revert if it breaks smth.

🔥 Rollback

Revert if it breaks smth.

📄 Procedure

Explain how the rollback for this change will look like, how we can recover fast.

🖥 Appliance

Note to reviewers: ensure that this change is compatible with the Appliance.

Copy link
Contributor

@VictorGarridoAuth0 VictorGarridoAuth0 left a comment

Choose a reason for hiding this comment

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

LGTM, tested on SUS

@nkavtur nkavtur merged commit 9662651 into master Feb 4, 2026
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

Development

Successfully merging this pull request may close these issues.

2 participants