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

[TP-2147] Add Gem Game contract #195

Merged
merged 10 commits into from
Apr 12, 2024
Merged

Conversation

pano-skylakis
Copy link
Contributor

@pano-skylakis pano-skylakis commented Apr 8, 2024

Adds a new contract - GemGame.

This contract includes a single function which emits an event. This event is intended to be indexed off-chain.

@pano-skylakis pano-skylakis marked this pull request as ready for review April 11, 2024 02:09
Comment on lines +15 to +25
interface IDeployer {
function deploy(bytes memory bytecode, bytes32 salt) external payable returns (address deployedAddress_);
function deployAndInit(bytes memory bytecode, bytes32 salt, bytes calldata init)
external
payable
returns (address deployedAddress_);
function deployedAddress(bytes calldata bytecode, address sender, bytes32 salt)
external
view
returns (address deployedAddress_);
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@drinkcoffee thoughts on moving this somewhere where it can be reused? Something else we could do is install the contract-deployer repository and potentially import from there 🤔

jeetparikh
jeetparikh previously approved these changes Apr 12, 2024
Copy link

@jeetparikh jeetparikh left a comment

Choose a reason for hiding this comment

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

Thanks Pano.. here is your oat milk ☕

Comment on lines +7 to +18
| Environment/Network | Deployment Address | Commit Hash |
|--------------------------|--------------------|-------------|
| imtbl-zkevm-testnet | - | - |
| imtbl-zkevm-mainnet | - | - |

# Status

Contract threat models and audits:

| Description | Date |Version Audited | Link to Report |
|---------------------------|------------------|-----------------|----------------|
| Not audited and no threat model | - | - | - |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will populate these in a follow-up PR

@pano-skylakis pano-skylakis merged commit 27d6570 into main Apr 12, 2024
6 of 7 checks passed
@drinkcoffee drinkcoffee deleted the ps/tp-2147/add-gems-game-contract branch April 26, 2024 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants