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

No longer working due to dependency on Request Module #28

Open
shak4031 opened this issue Oct 7, 2021 · 5 comments
Open

No longer working due to dependency on Request Module #28

shak4031 opened this issue Oct 7, 2021 · 5 comments

Comments

@shak4031
Copy link

shak4031 commented Oct 7, 2021

This module is no longer working due to dependencies on deprecated modules like "request".

@qistoph
Copy link
Owner

qistoph commented Oct 8, 2021

I'm currently unable to work on an update. A PR would be most welcome

@fribse
Copy link

fribse commented Feb 13, 2022

I see this as well with the new 2.18 of MM (and thus the update of node.js):

0|MagicMirror  | [13.02.2022 12:48.21.931] [ERROR] App threw an error during load
0|MagicMirror  | [13.02.2022 12:48.21.940] [ERROR] Error: Cannot find module 'request'
0|MagicMirror  | Require stack:
0|MagicMirror  | - /home/pi/MagicMirror/modules/MMM-MyCommute/node_helper.js
0|MagicMirror  | - /home/pi/MagicMirror/js/app.js
0|MagicMirror  | - /home/pi/MagicMirror/js/electron.js
0|MagicMirror  | - /home/pi/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
0|MagicMirror  | -
0|MagicMirror  |     at Module._resolveFilename (node:internal/modules/cjs/loader:940:15)
0|MagicMirror  |     at Function.n._resolveFilename (node:electron/js2c/browser_init:249:1128)
0|MagicMirror  |     at Function.Module._resolveFilename (/home/pi/MagicMirror/node_modules/module-alias/index.js:49:29)
0|MagicMirror  |     at Module._load (node:internal/modules/cjs/loader:785:27)
0|MagicMirror  |     at Function.c._load (node:electron/js2c/asar_bundle:5:13331)
0|MagicMirror  |     at Module.require (node:internal/modules/cjs/loader:1012:19)
0|MagicMirror  |     at require (node:internal/modules/cjs/helpers:94:18)
0|MagicMirror  |     at Object.<anonymous> (/home/pi/MagicMirror/modules/MMM-MyCommute/node_helper.js:9:17)
0|MagicMirror  |     at Module._compile (node:internal/modules/cjs/loader:1116:14)
0|MagicMirror  |     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1169:10)
0|MagicMirror  | [13.02.2022 12:48.21.945] [ERROR] Whoops! There was an uncaught exception...
0|MagicMirror  | [13.02.2022 12:48.21.968] [ERROR]
0|MagicMirror  | Error: Cannot find module 'request'

Sigh, another dead module.

@qistoph
Copy link
Owner

qistoph commented Feb 13, 2022

A quick fix is probably to install requests yourself. As said before, a PR is most welcome.

The great thing with open source is that you can (easily) fix any issues. As times change people aren't always as committed to keeping stuff up-to-date. Also, this module has been running on my mirror for years and still is.

I'll try to look into this issue, but don't have the time I used to have for such hobby projects and have always hoped that sharing my work would encourage others to contribute. The increasing amount of complaints and requests is making me reconsider

@fribse
Copy link

fribse commented Feb 14, 2022

No problem, just so sad, but I do understand the time constraints, I live with them every day 😄

@fribse
Copy link

fribse commented Feb 14, 2022

I got it working today byu adding the request package, so that's progress :-)

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

No branches or pull requests

3 participants