-
Notifications
You must be signed in to change notification settings - Fork 203
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
Update links, content and code block in deserialize custom data frontend course #402
base: main
Are you sure you want to change the base?
Update links, content and code block in deserialize custom data frontend course #402
Conversation
6853aac
to
658e1ac
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thnaks @khankluan24 ! Good, but mainly seems to be updating for findProgramAddressSync()
. I've left a few questions and comments above. Not there are multiple people competing for this lesson so competition is hot!
you need a refresher on either of these, have a look at the previous lesson. | ||
Just as before, we’ll use [Borsh](https://github.com/near/borsh) and | ||
`@coral-xyz/borsh`. If you need a refresher on either of these, have a look at | ||
the previous lesson. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I saw the Borsh website was down, but it's back up again - https://borsh.io/
@@ -247,7 +248,7 @@ structured. | |||
#### 3. Create a method to deserialize data | |||
|
|||
Now that we have the buffer layout set up, let’s create a static method in | |||
`Movie` called `deserialize` that will take an optional `Buffer` and return a | |||
`Movie` called `deserialize` that will take an optional `buffer` and return a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
`Movie` called `deserialize` that will take an optional `buffer` and return a | |
`Movie` called `deserialize` that will take an optional `Buffer` and return a |
The type in node is Buffer
capital B
@@ -377,7 +378,6 @@ If you get really stumped, feel free to | |||
As always, get creative with these challenges and take them beyond the | |||
instructions if you want! | |||
|
|||
<Callout type="success" title="Completed the lab?"> | |||
Push your code to GitHub and | |||
Push your code to GitHub and <Callout type="success" title="Completed the lab?"> | |||
[tell us what you thought of this lesson](https://form.typeform.com/to/IPH0UGz7#answers-lesson=9cb89e09-2c97-4185-93b0-c89f7aca7677)! | |||
</Callout> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Re: the images, is the only change adding the ...Sync()
?
Problem
Update links, content and code block in deserialize custom data frontend course
Summary of Changes
web3.PublicKey.findProgramAddress -> web3.PublicKey.findProgramAddressSync
due to deprecation).