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

Final TypeScript changes #337

Merged
merged 5 commits into from
Apr 7, 2021
Merged

Final TypeScript changes #337

merged 5 commits into from
Apr 7, 2021

Conversation

jdtzmn
Copy link
Contributor

@jdtzmn jdtzmn commented Apr 7, 2021

  • Remove deprecated ESLint setting

  • Specify SuggestedParams type for the SuggestedParams client request

  • Fix default generic types of the JSONRequest class

  • Provide better return type for the assignGroupID method

  • Export Account type

Closes #336.

* Remove deprecated ESLint setting

* Specify `SuggestedParams` type for the SuggestedParams client request

* Fix default generic types of the `JSONRequest` class

* Provide better return type for the `assignGroupID` method

* Export `Account` type
@jdtzmn jdtzmn requested a review from jasonpaulos April 7, 2021 16:10
@jdtzmn jdtzmn force-pushed the jacob/typescript-final-changes branch from c46ac77 to 86e372d Compare April 7, 2021 20:21
@jdtzmn jdtzmn force-pushed the jacob/typescript-final-changes branch from 86e372d to 5d0e06c Compare April 7, 2021 20:25
src/makeTxn.ts Outdated Show resolved Hide resolved
* Make `makeKeyRegistrationWithSuggestedParamsFromObject` function overloads stricter
@jdtzmn jdtzmn force-pushed the jacob/typescript-final-changes branch from 1b91d9d to dea9ba5 Compare April 7, 2021 21:14
Copy link
Contributor

@jasonpaulos jasonpaulos left a comment

Choose a reason for hiding this comment

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

Pending tests, looks great!

@jdtzmn jdtzmn merged commit 18f911b into develop Apr 7, 2021
@jdtzmn jdtzmn deleted the jacob/typescript-final-changes branch April 7, 2021 21:55
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.

Final TypeScript Changes
2 participants