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

[Meteor 3] Update package to async #149

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

sebastianspiller
Copy link

Hello there!

I made some changes to make synced cron work with async. I also translated it to es6, which I will add in a future pr, so that we can discuss changes in smaller steps.

In tiny test, I still get errors with, so it is just a wip pr:

  • exception - message Cannot await without a Fiber
  • exception - message Meteor code must always run within a Fiber. Try wrapping callbacks that you pass to non-Meteor libraries with Meteor.bindEnvironment.

It would be nice, if you have a look, thank you.

Regards
Sebastian

@Grubba27 Grubba27 changed the title wip: updated to async, but tests still fail [Meteor 3] Update package to async Dec 4, 2023
Copy link

@dallman2 dallman2 left a comment

Choose a reason for hiding this comment

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

I am doing the same thing to allow my Meteor 3 app to run. Please merge these changes.

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.

None yet

3 participants