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

cleaning up Chapter 3 #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ryancwalsh
Copy link

I'm still trying to understand the crossword puzzle tutorial. So I'm going through it again and testing the instructions.

One problem with the Readme was that it showed a command with a deposit, but this Chapter 3 contract's new_puzzle method doesn't expect a deposit because it's using:

// 5 Ⓝ in yoctoNEAR
const PRIZE_AMOUNT: u128 = 5_000_000_000_000_000_000_000_000;

Even when changing the new_puzzle command to not offer a deposit, I get this error:

"Smart contract panicked: panicked at 'Failed to deserialize input from JSON.: Error("missing field args", line: 1, column: 68)', src/lib.rs:133:1"}}

So more updates to this Readme (or the contract) are still required before this lesson actually works.

I'm still trying to understand the crossword puzzle tutorial. So I'm going through it again and testing the instructions.

One problem with the Readme was that it showed a command with a deposit, but this Chapter 3 contract's `new_puzzle` method doesn't expect a deposit because it's using:

```
// 5 Ⓝ in yoctoNEAR
const PRIZE_AMOUNT: u128 = 5_000_000_000_000_000_000_000_000;
```

Even when changing the `new_puzzle` command to not offer a deposit, I get this error:

> "Smart contract panicked: panicked at 'Failed to deserialize input from JSON.: Error(\"missing field `args`\", line: 1, column: 68)', src/lib.rs:133:1"}}

So more updates to this Readme (or the contract) are still required before this lesson actually works.
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.

1 participant