-
Notifications
You must be signed in to change notification settings - Fork 0
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
error fetch #1
Comments
You need node v18 or newer. I assumed people usually install node from the FlowLauncher settings. I‘d prefer to not import another node-module to support older versions because some other modules also require v18, but I‘ll add a version check and appropriate error message. |
Oh wow I just checked that and Flow Launcher uses a version old enough to have reached EOL by default. Somebody already filed a bug report for that here. I will still look into adding node-fetch sometime after christmas :) |
Thx dude ! |
This should be fixed in the latest version, which will be available in the Plugin Store shortly :) |
hi
I'm testing your addon which interests me a lot.
but I have an error as you see below
const response = await fetch(apiUrl);
ReferenceError: fetch is not defined
I think I found the error
it's because the base node doesn't take fetch
I solved your problem by installing node-fetch with
npm install node-fetch
and adding to line 10 in main.js:
import fetch from 'node-fetch';
with this node understands fetch
I'll create a bug-fetch branch and push you the code
thanks for your work on this addon!
now with my change
The text was updated successfully, but these errors were encountered: