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

Migrated Account Functions to use Js SDK #609

Merged
merged 2 commits into from
Apr 6, 2021
Merged

Conversation

@tomijaga
Copy link
Author

tomijaga commented Apr 1, 2021

Hey, @angle943 how do you want to review the code?
Do u want to review and merge each change or could you give me access to merge then you could review the code all at once when I'm done

@zinoadidi
Copy link

@angle943 maybe you can assign one of the other contributors/team members to review for us, then you only need to check it when we are ready to merge to development

@angle943
Copy link
Contributor

angle943 commented Apr 6, 2021

@zinoadidi i gave you write permissions in this repo. After you've reviewed, feel free to merge it to the javascript-sdk-migration branch. Once that branch is fully ready, i'll review it

const block = await createBlock(primaryValidatorConfig, senderSigningKey, senderAccountNumber, txs);
const sender = new Account(senderSigningKey);
const balanceLock = await fetchAccountBalanceLock(senderAccountNumber, primaryValidatorConfig);
const block = sender.createBlockMessage(balanceLock, txs);
await axios.post(`${address}/blocks`, block, {
Copy link

@zinoadidi zinoadidi Apr 6, 2021

Choose a reason for hiding this comment

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

@tomijaga The post requests here will be handled by sdk in later fix right ?
Or how is it suppose to work; I believe all interactions with chain will go through us eventually

Copy link
Author

Choose a reason for hiding this comment

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

It would be handled when I implement the payment handler class

@zinoadidi
Copy link

🥳

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.

3 participants