-
Notifications
You must be signed in to change notification settings - Fork 12
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
Make sure Adonis/Addons/Throttle does have a extend method. Report this issue to the provider author > More details: https://err.sh/adonisjs/errors/E_INVALID_IOC_MANAGER #11
Comments
@AndrewJo any ideas? |
It's working before but now it's not. Maybe because of adonis update? |
I came across this earlier today too. I've been able to fix it locally so I've created a pull request that provides the fix and updates the associated test. I've provided more detail in the PR: #12 Let's await feedback from the maintainers to see whether this PR is appropriate and whether they agree to merge it in. Many thanks! |
Oof. Just saw this now (just flew out for my wedding so haven't been checking my email) My bad guys. I should've tested it more thoroughly with a sample AdonisJs project. I'll review @Barryld's PR #12 and perhaps set up an integration test so we can catch things like this before it makes it into the codebase. Again, sorry and I take full responsibility for this bug. |
I've merged the fix, should be okay now 👍 |
InvalidArgumentException: E_INVALID_IOC_MANAGER: Make sure Adonis/Addons/Throttle does have a extend method. Report this issue to the provider author
at /Users/eco/Code/api.jericopulvera.com/server.js(anonymous):24
19
20 const { Ignitor } = require('@adonisjs/ignitor')
21
22 new Ignitor(require('@adonisjs/fold'))
23 .appRoot(__dirname)
24 .fireHttpServer()
25 .catch(console.error)
The text was updated successfully, but these errors were encountered: