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

Trying to run at desk #144

Closed
Ecleptic opened this issue Sep 14, 2022 · 11 comments · Fixed by #146
Closed

Trying to run at desk #144

Ecleptic opened this issue Sep 14, 2022 · 11 comments · Fixed by #146
Assignees
Labels
bug Something isn't working

Comments

@Ecleptic
Copy link

Ecleptic commented Sep 14, 2022

When running the application, I get this error and googling does not seem to help me get around it.

{
  name: 'NoSuchKey',
  message: 'The specified key does not exist.',
  status: 500,
  stack: 'NoSuchKey: The specified key does not exist.\n' +
    '    at deserializeAws_restXmlNoSuchKeyResponse (~/projects/notea/node_modules/@aws-sdk/client-s3/dist-cjs/protocols/Aws_restXml.js:6154:23)\n' +
    '    at deserializeAws_restXmlGetObjectCommandError (~/projects/notea/node_modules/@aws-sdk/client-s3/dist-cjs/protocols/Aws_restXml.js:4353:25)\n' +
    '    at runMicrotasks (<anonymous>)\n' +
    '    at processTicksAndRejections (internal/process/task_queues.js:97:5)\n' +
    '    at async ~/projects/notea/node_modules/@aws-sdk/middleware-serde/dist-cjs/deserializerMiddleware.js:7:24\n' +
    '    at async ~/projects/notea/node_modules/@aws-sdk/middleware-signing/dist-cjs/middleware.js:11:20\n' +
    '    at async StandardRetryStrategy.retry (~/projects/notea/node_modules/@aws-sdk/middleware-retry/dist-cjs/StandardRetryStrategy.js:51:46)\n' +
    '    at async ~/projects/notea/node_modules/@aws-sdk/middleware-flexible-checksums/dist-cjs/flexibleChecksumsMiddleware.js:56:20\n' +
    '    at async ~/projects/notea/node_modules/@aws-sdk/middleware-logger/dist-cjs/loggerMiddleware.js:6:22\n' +
    '    at async StoreS3.getObject (webpack-internal:///(api)/./libs/server/store/providers/s3.ts:82:28)'
}
@tecc tecc added the bug Something isn't working label Sep 14, 2022
@tecc
Copy link
Member

tecc commented Sep 14, 2022

That's interesting. What object storage software are you using?
I'm guessing it could be fixed by changing line 18 in libs/server/store.ts to include err.name, but I may be entirely wrong.

@Ecleptic
Copy link
Author

All I've done is the steps in the readme:
docker-compose up -d
yarn dev

I assume it's a volume permission issue? I am not terribly familiar with docker.

@Ecleptic
Copy link
Author

changing line 18 to include err.name did help me create a file, however typing in the body got me a new error. Which I am looking up now.
SCR-20220914-lhi

@Ecleptic
Copy link
Author

What version of node do you use?

@tecc
Copy link
Member

tecc commented Sep 14, 2022

I use node 16.

@Ecleptic
Copy link
Author

That wasn't it. 😆

@Ecleptic
Copy link
Author

ueberdosis/tiptap#577

This error references the issue. and this did work when I did a yarn install

@Ecleptic
Copy link
Author

we may need to update node packages to address this.

@tecc
Copy link
Member

tecc commented Sep 14, 2022

Perhaps. I'll have to properly look into it later, since I'm slightly busy at the moment.

tecc added a commit that referenced this issue Sep 17, 2022
NoSuchKey: I've observed errors to have the type property named `name` instead of `code` or `message`. See #144.
@tecc
Copy link
Member

tecc commented Sep 22, 2022

Okay, update on this issue: Since I can't test it myself at this moment, I wonder if downgrading prosemirrror-model to ^1.8.1 - I perused a bit through the lockfiles of both rich-markdown-editor and Notea, and it might be caused by having a too new version for the editor library.
The main reason I'm suggesting this is that rich-markdown-editor has been archived for nearly a year, and as such may not work with the later versions of Prosemirror.
If not that, it might be that the prosemirror versions used are incompatible with each other.
This is all speculative though.

@tecc tecc self-assigned this Sep 22, 2022
tecc added a commit that referenced this issue Sep 24, 2022
144: The version of prosemirror automatically resolved was incompatible with the current editor backend (rich-markdown-editor), so it had to be manually specified in the "resolutions" field of package.json. Fixes #144.
@tecc
Copy link
Member

tecc commented Sep 24, 2022

Quick update on this issue: I tested it with the changes I made in #146, and it works fine.

@tecc tecc closed this as completed in #146 Sep 24, 2022
tecc added a commit that referenced this issue Sep 24, 2022
#144: Manually specified version of prosemirror-mdoel
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants