Skip to content

Conversation

GiovaniGuizzo
Copy link
Contributor

Checklist for Pull Requests

  • All tests pass (yarn test:all)
  • Code follows the style guide and passes lint checks
  • Documentation is updated (README, docs, etc)
  • Linked to corresponding issue, if applicable

Summary of Changes

Closes #59

This does not solve the problem if one is running Sidequest as ESM and trying to enqueue a job from within a CJS job.
Also does not work the other way around - running Sidequest as CJS and trying to enqueue a job from within an ESM job.

…formation

fix: ensure backend is set correctly in Sidequest.configure method

chore: add comments for dynamic import replacements in Rollup config for CJS compatibility
@GiovaniGuizzo GiovaniGuizzo requested a review from merencia as a code owner August 6, 2025 21:36
merencia
merencia previously approved these changes Aug 7, 2025
@GiovaniGuizzo GiovaniGuizzo merged commit 14c959c into master Aug 7, 2025
2 checks passed
@GiovaniGuizzo GiovaniGuizzo deleted the fix-job-cjs branch August 7, 2025 23:32
sidequest-release bot pushed a commit that referenced this pull request Aug 7, 2025
## [1.4.1](v1.4.0...v1.4.1) (2025-08-07)

### Bug Fixes

* fixes job enqueue inside CJS jobs ([#60](#60)) ([14c959c](14c959c))
@sidequest-release
Copy link

🎉 This PR is included in version 1.4.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enqueueing jobs from inside other jobs does not work with CJS
2 participants