Programming Project - Phase 2

dan-i · February 18, 2021, 3:03pm

Can you push your project to GitHub? I want to try to deploy it and see if the error is somewhere in your project or in the pkg you have installed

Ping me once pushed to GitHub, I will take a look

Jose_Hernandez · February 18, 2021, 4:47pm

Sorry, do you have a good recommendation on what to watch regarding how to use github? Or would it be easier to just drag and drop the files?

Thanks again!

NamaWho · February 18, 2021, 5:02pm

Hi, I think you just missed a ; after event LogNewProvableQuery…

Jose_Hernandez · February 18, 2021, 5:03pm

Hi! There is just a missing semi-colon at the end of this line. Hope you have smooth sailing from here

Jose_Hernandez · February 19, 2021, 6:44am

https://github.com/jlthcrypto/coin-flip

Hope I did it right

chim4us · February 18, 2021, 9:42pm

@dan-i You don’t understand my problem
I want to run npm init on bet project path. so that folder node_modules will be created as suggested by filip.
It ran successfully but the path node_modules was not created.
I copied the path node_modules from my old project and run npm install truffle-hdwallet-provider in order to install the wallet provider. it throws the error

After i done what you suggested it came with error

kHarsh · February 19, 2021, 12:04am

@dan-i the code that is shown above is not working for me, I am scratching my head for couple of days now but latest number I am getting back is always 0.

Can you please take a look at the code again and tell me if I am missing something. On UI side I make sure the abi and network address is the latest deployed so no issues there.

dan-i · February 19, 2021, 9:54am

Hey @Jose_Hernandez

Thanks for uploading

I tried to deploy your contract and I get this error:

Although you don’t see the error message, the error location looks the same.

In your previous post:
at Object.run ([.......]**packages/migrate/index.js:96:1**)

Which is the same in my screenshot.

Let’s analyse the error together

Notice that the error happens during the migration of 1_initial_migration.js;
Notice the error message This function is restricted to the contract's owner

So let’s just follow the clues:

Open the file 1_initial_migration.js
You are migrating the project from account[1] that is not the owner. The owner is always accounts[0].
Notice that you should never touch the file 1_initial_migration.js, it is created and used by the system and you must leave it as it is.

Just cancel your modification and set is back at its initial state:

const Migrations = artifacts.require("Migrations");

module.exports = function(deployer) {
  deployer.deploy(Migrations);
};

This fixes the issue.

You have another issue in your 2nd migration file 2_coinFlip_migration.js
Let’s see if you can spot the issue by yourself.

If you need help I am here so no worries!

Happy coding,
Dani

dan-i · February 19, 2021, 9:57am

Hi @chim4us

There are no errors in this screenshots, just warnings that you can ignore.

Also in your screenshot above, I see a package.json file, so I don’t understand what’s wrong

Can you send a screenshot of your truffle project directory where all the files are visible?

Thanks!
Dani

chim4us · February 19, 2021, 10:24am

Hi @dan-i

How can i ignore them. Please find attach as request

Best Regards
Chimguy

NamaWho · February 19, 2021, 10:26am

Still stuck with this issue
I pushed on GitHub my project, even though it’s still at an early stage. Still needs lots of changes, but in this way I think you can figure it out better. CoinFlip.sol

@dan-i could you spot the error?

Sorry to bother you. Thanks!
Daniel

Jose_Hernandez · February 19, 2021, 10:54am

Thank you so much! Also removed the json object in 2_coinFlip_migration.js

So are we always tied to using accounts[0] as the owner? Or is there a way to choose a different one like in Ganache?

dan-i · February 19, 2021, 11:11am

Accounts[0] is the owner by default

Cheers,
Dani

dan-i · February 19, 2021, 11:12am

Hi @chim4us

From your screenshot I do not see any error, you can move forward and keep coding.

regards,
Dani

dan-i · February 19, 2021, 11:13am

Hey @NamaWho

@dan-i could you spot the error? is a bit too generic
Can you please describe the issue? What happens, when?

thanks!
Dani

NamaWho · February 19, 2021, 12:01pm

I think I’ve described it here

Anyway, if it sounds still too generic I’ve got no problem trying to explain myself better.
The fact is that even though I have a mapping where I match “betId” -> “playerAddress” (in placeBet()), then when I try to read the same record in closeBet() it seems it doesn’t exist. You can see this behavior from the events that are emitted in both functions, which I pasted in the previous comment.

enrico · February 19, 2021, 12:21pm

hello,

I finally finished my projects

this is the project made with ganache: https://github.com/enricofabris/COIN_CONTRACT-GANACHE

this is the project made with provable oracle and ropsten: https://github.com/enricofabris/COIN_CONTRACT---ORACLE---ROPSTEN

I tried both of them and they seem to work.
Could you please let me know some feedbak?

besides I was able to use the test only with the first project because in the course it is shown to use the test through console.
But in the second project when we use ropsten how can we do the test of the contract?

thanks

chim4us · February 19, 2021, 12:22pm

@filip I did not see provableAPI.sol you mention on your lesson

dan-i · February 19, 2021, 1:13pm

Hey @NamaWho

Can you please post the whole contract? I am interested in checking the function update().
Let me know

Cheers,
Dani

NamaWho · February 19, 2021, 1:43pm

Course, you can find it here.

P.S. I changed a bit those functions from yesterday but the logic is still the same; the error too