Skip to content
This repository was archived by the owner on Apr 20, 2023. It is now read-only.

Conversation

@cuteolaf
Copy link
Contributor

No description provided.

Copy link
Member

@f-gate f-gate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for the sake of transparency i havent looked at all the files

@samelamin
Copy link
Contributor

@cuteolaf can you merge imbue enterprise into this branch and fix conflicts

Also I tried running this branch and received a few errors, most likely due to lint?

    return new TSError(diagnosticText, diagnosticCodes, diagnostics);
           ^
TSError: ⨯ Unable to compile TypeScript:
src/backend/middleware/authentication/strategies/web3/polkadot-js.ts:36:3 - error TS7006: Parameter 'jwt_payload' implicitly has an 'any' type.

36   jwt_payload,
     ~~~~~~~~~~~
src/backend/middleware/authentication/strategies/web3/polkadot-js.ts:37:3 - error TS7006: Parameter 'next' implicitly has an 'any' type.

37   next
     ~~~~

    at createTSError (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:859:12)
    at reportTSError (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:863:19)
    at getOutput (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:1077:36)
    at Object.compile (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:1433:41)
    at Module.m._compile (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:1617:30)
    at Module._extensions..js (node:internal/modules/cjs/loader:1272:10)
    at Object.require.extensions.<computed> [as .ts] (/Users/samelamin/src/dapp/api/node_modules/ts-node/src/index.ts:1621:12)
    at Module.load (node:internal/modules/cjs/loader:1081:32)
    at Function.Module._load (node:internal/modules/cjs/loader:922:12)
    at Module.require (node:internal/modules/cjs/loader:1105:19) {
  diagnosticCodes: [ 7006, 7006 ]
}

@cuteolaf cuteolaf marked this pull request as draft March 20, 2023 18:40
Sign up for free to 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.

4 participants