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

Feature/overall-improvements #3

Merged
merged 5 commits into from
Oct 8, 2023

Conversation

oscarpascualbakker
Copy link
Owner

Add github actions file

-Adding collection name to Contract
-Adding Slither to whole project
-Adding proper container names in Docker
-Setting last compiler version
-Solving some Slither discoveries
-Adding NFT image to repo
-Testing with Alchemy and PHP (wip)
-Adding more tests
-Add ID in mint function to allow more than one NFT
-Add special requires to mint function
-Fix vote function problem
-Include collection name in contract for certain wallets
-Install Web3 component to change evm_timestamp for testing
-Add more tests
-Update README.md
Add github actions file
@oscarpascualbakker oscarpascualbakker merged commit 445bf53 into master Oct 8, 2023
1 check failed
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