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

feat(vara): add TransactionPaymentCallApi #3393

Merged
merged 2 commits into from
Oct 9, 2023
Merged

Conversation

ukint-vs
Copy link
Member

@ukint-vs ukint-vs commented Oct 7, 2023

Resolves # .

@reviewer-or-team

@ukint-vs ukint-vs added the A0-pleasereview PR is ready to be reviewed by the team label Oct 7, 2023
@ukint-vs ukint-vs requested review from ekovalev and breathx October 7, 2023 05:59
@ukint-vs ukint-vs self-assigned this Oct 7, 2023
@ukint-vs ukint-vs force-pushed the vs/vara-tx-call-api branch from 6c91d79 to a254ad4 Compare October 7, 2023 09:16
@ukint-vs ukint-vs force-pushed the vs/vara-tx-call-api branch from a254ad4 to c80e4d5 Compare October 7, 2023 09:23
@ekovalev
Copy link
Member

ekovalev commented Oct 7, 2023

What's wrong with https://github.com/gear-tech/gear/blob/master/runtime/common/src/apis.rs#L130?

@breathx, we definitely need to merge the runtime api implementations in one place. It very much has to do with the #3380, maybe you could make a quick fix for this in that PR, as well?

@ukint-vs
Copy link
Member Author

ukint-vs commented Oct 7, 2023

What's wrong with https://github.com/gear-tech/gear/blob/master/runtime/common/src/apis.rs#L130?

This is one of the 2 tx payment apis. One is general and second is CallApi.

@ekovalev
Copy link
Member

ekovalev commented Oct 8, 2023

What's wrong with https://github.com/gear-tech/gear/blob/master/runtime/common/src/apis.rs#L130?

This is one of the 2 tx payment apis. One is general and second is CallApi.

Ok, got it. Then it's even more logical to move the "common" apis back to Vara runtime apis.

@ekovalev ekovalev closed this Oct 8, 2023
@ekovalev ekovalev reopened this Oct 8, 2023
@ukint-vs ukint-vs merged commit 77d537f into master Oct 9, 2023
20 checks passed
@ukint-vs ukint-vs deleted the vs/vara-tx-call-api branch October 9, 2023 05:44
@shamilsan shamilsan added the B1-releasenotes The feature deserves to be added to the Release Notes label Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview PR is ready to be reviewed by the team B1-releasenotes The feature deserves to be added to the Release Notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants