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

Add types #19

Merged
merged 1 commit into from
Nov 7, 2021
Merged

Add types #19

merged 1 commit into from
Nov 7, 2021

Conversation

Creskendoll
Copy link
Contributor

Greetings, I've been experimenting with Svelte and this library and thought that types must be added sooner or later, so here it is...

Added .d.ts types complete with docstrings copied from the readme.

Even though defaultChainStore contains the properties listed in the ChainStore, the API is not intended to be used that way. Instead, the store properties are exported separately and not exposed from within DefaultChainStore. The properties are also included in ChainStore so that they can be accessed the way the docs suggest,

Cheers

@clbrge
Copy link
Owner

clbrge commented Nov 7, 2021

Thanks @Creskendoll . I don't use typescript myself so I will trust your judgement on these types.

@clbrge clbrge merged commit 54c3799 into clbrge:master Nov 7, 2021
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.

2 participants