brianwchou / hangman Goto Github PK

Use https://sequencediagram.org

With the currently deployed HangmanFactory from PR #76, we currently do not have a user flow for accessing User's link tokens.

We need to figure out how to deploy and test on a testnet. All need all tests to be modified/copied into their testnet form so that we can test on testnet.

This means remove mocking
This means linking to existing contracts
This means using all testnet values

• chainlinkTokenAddress
• chainlinkOracleAddress
• chainlinkHttpGetJobId

the contact needs to be given an allowance from the caller's address in order to move link to pay LINK to the link token contract.

Seems Chainlink finally updated to 0.5.x
https://github.com/smartcontractkit/chainlink/tree/develop/evm/v0.5

https://github.com/brianwchou/Hangman/blob/9425937b5adac64db94aa100b90d1913a190bbf4/contracts/StartGame.sol#L37

I'm not sure if I like having to do the computation first as a call then a transaction. This means that either we do a call and then a transaction, or we do a transaction and emit the value. I think this depends on how dependable emit can be and if it can be caught reliably. If we do the emit, we probably need to add an identifier as to ensure we're listening to the right emitted event.

update url usage and the constructor parameter

Create a writeup that describes the process and thinking for mocking up the chainlink contract for up coming medium article.

eslint

https://www.npmjs.com/package/eth-gas-reporter

depending on main or test net
https://docs.chain.link/docs/decentralized-oracles-ethereum-mainnet

Deploy the HangmanFactory.sol to mainnet

constructor(address _link, address _oracle, string _url, string[] _path)