-
Notifications
You must be signed in to change notification settings - Fork 6
Added typescript definitions #26
base: master
Are you sure you want to change the base?
Conversation
Add JS implementation of Indy methods
Add implementation of iOS Indy methods
Ignoring it globally causes android src to be excluded from the packed tarball Signed-off-by: Gaurav Narula <[email protected]>
npmignore: ignore src only in project root
Signed-off-by: Jakub Koci <[email protected]>
* Store pool and wallet handles in maps Makes the API compatible with libindy nodejs wrapper * Update exported methods to use walletHandle and buffers iOS native calls remain untested * android: Update module name * crypto: fix return types Signed-off-by: Gaurav Narula <[email protected]>
Signed-off-by: Gaurav Narula <[email protected]>
Add non_secrets API
Signed-off-by: Gaurav Narula <[email protected]>
Bump indy version
Rename IndyBridge to IndySdk (#10)
Bumps [lodash](https://github.com/lodash/lodash) from 4.17.15 to 4.17.19. - [Release notes](https://github.com/lodash/lodash/releases) - [Commits](lodash/lodash@4.17.15...4.17.19) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Added implementation of buildGetAttribRequest, buildGetNymRequest and parseGetNymResponse Signed-off-by: Gaurav Narula <[email protected]>
Signed-off-by: Timo Glastra <[email protected]>
Hi @riccardocaranfil! We're working on getting the types published to the DefinitelyTyped repo. Because indy-sdk and rn-indy-sdk share the same type definitions, we would have to maintain two versions of the definition if we add them to the repo. See https://github.com/TimoGlastra/DefinitelyTyped/blob/%40types/indy-sdk/types/indy-sdk/index.d.ts for the current working version. It does seem like your definition contains some types not in our version and vice versa, so it'll be good to combine them. See openwallet-foundation/credo-ts#38 for more informatin |
@TimoGlastra sounds good! Thanks! |
@riccardocaranfil I finally got around to creating a PR in the definitely typed repo. See https://www.npmjs.com/package/@types/indy-sdk for the published package. You can use it for # npm
npm install @types/rn-indy-sdk@npm:@types/indy-sdk
# yarn
yarn add @types/rn-indy-sdk@npm:@types/indy-sdk Or adding it to your {
//...
"devDependencies": {
"@types/rn-indy-sdk": "npm:@types/indy-sdk"
}
} I see your PR contains types that are not in the repo yet. I'd like to also get them included in the See https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/indy-sdk |
Just to avoid @ts-ignore in typescript project