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

wrapInTransaction infer to the original function #21

Merged
merged 1 commit into from
Nov 30, 2022

Conversation

lytc
Copy link

@lytc lytc commented Nov 28, 2022

No description provided.

@Aliheym
Copy link
Owner

Aliheym commented Nov 29, 2022

Hi, thanks for your PR. Maybe it will be better to add infer for function arguments too?

@lytc
Copy link
Author

lytc commented Nov 29, 2022

@Aliheym It already does!
Untitled

@Aliheym
Copy link
Owner

Aliheym commented Nov 29, 2022

@Aliheym It already does!
Untitled

Cool, thanks

@Aliheym Aliheym merged commit a8cfaf5 into Aliheym:master Nov 30, 2022
@lytc
Copy link
Author

lytc commented Dec 1, 2022

@Aliheym Thank you so much for approved the PR. Could you also help release this fix as well?

@Aliheym
Copy link
Owner

Aliheym commented Dec 1, 2022

@lytc done.

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.

None yet

2 participants