Skip to content

Conversation

@confleux
Copy link
Member

@confleux confleux commented Jun 10, 2025

  • Updated dev deps and others
  • Updating express caused issue, which i resolved:
src/api/file.ts:13:54 - error TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type '(req: Request<ParamsDictionary, any, any, ParsedQs, Record<string, any>>, res: Response<any, Record<string, any>, number>) => Promise<...>' is not assignable to parameter of type 'RequestHandlerParams<ParamsDictionary, any, any, ParsedQs, Record<string, any>>'.
      Type '(req: Request<ParamsDictionary, any, any, ParsedQs, Record<string, any>>, res: Response<any, Record<string, any>, number>) => Promise<...>' is not assignable to type 'RequestHandler<ParamsDictionary, any, any, ParsedQs, Record<string, any>>'.
        Type 'Promise<Response<any, Record<string, any>, number> | undefined>' is not assignable to type 'void | Promise<void>'.
          Type 'Promise<Response<any, Record<string, any>, number> | undefined>' is not assignable to type 'Promise<void>'.
            Type 'Response<any, Record<string, any>, number> | undefined' is not assignable to type 'void'.
              Type 'Response<any, Record<string, any>, number>' is not assignable to type 'void'.

13 router.post('/upload', multerStorage.array('files'), async (req, res) => {
  • Updating eslint to v9 forced to use flat config instead of .eslintrc.json. We've used to have gts, but it's not updated to support eslint v9, so I've created new config and removed gts.
  • Updating helia caused issue with renaming connectionEncryption to connectionEncrypters, which I've resolved
  • Updating unixfs caused issue with method addFile() wrapping a file into a dir and returning dir's cid instead of file's, so I've used method addByteStream()

@adamant-al adamant-al requested a review from a team June 12, 2025 15:38
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

Successfully merging this pull request may close these issues.

3 participants