Skip to content
This repository was archived by the owner on Aug 18, 2025. It is now read-only.

Conversation

@vmauge
Copy link

Some apps do not have a media dir under static/apps/<app_name> for
example eyeball and hello_coder. When you try to edit the apps with app
editor, there is an error display in the consol. The fix checks if the
folder exists before reading it.

Error in the conle is :

16 Sep 23:20:46 - GET: edit/hello_coder editor 16 Sep 23:20:47 - GET: api/metadata/get/hello_coder editor 16 Sep 23:20:47 - GET: api/media/list/hello_coder editor Error: ENOENT, no such file or directory '/Users/vincent/dev/coder/coder-base/static/apps/hello_coder/media/' at Object.fs.readdirSync (fs.js:654:18) at Object.exports.listMedia (/Users/vincent/dev/coder/coder-base/apps/editor/app.js:338:20) at Object.exports.api_media_list_handler (/Users/vincent/dev/coder/coder-base/apps/editor/app.js:107:21) at apphandler (/Users/vincent/dev/coder/coder-base/server.js:111:46) at /Users/vincent/dev/coder/coder-base/server.js:255:60 at callbacks (/Users/vincent/dev/coder/coder-base/node_modules/express/lib/router/index.js:161:37) at param (/Users/vincent/dev/coder/coder-base/node_modules/express/lib/router/index.js:135:11) at pass (/Users/vincent/dev/coder/coder-base/node_modules/express/lib/router/index.js:142:5) at Router._dispatch (/Users/vincent/dev/coder/coder-base/node_modules/express/lib/router/index.js:170:5) at Object.router (/Users/vincent/dev/coder/coder-base/node_modules/express/lib/router/index.js:33:10) 16 Sep 23:20:47 - GET: api/getcode/hello_coder editor 

Some apps do not have a media dir under static/apps/<app_name> for example eyeball and hello_coder. When you try to edit the apps with app editor, there is an error display in the consol. The fix checks if the folder exists before reading it.
@jmstriegel
Copy link
Contributor

all apps need, and should have at installation, a media directory. i think some were wiped by accident since git ignores empty directories. i'm doing some restructuring that should eliminate this problem.

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vmauge@jmstriegel